Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Move to integration domain key

See original GitHub issue

According to this ADR 0007, we should move mitemp_bt to the integration domain key.

New integrations have to put the integration configuration section under the integration domain key if configuration YAML is used.

If I understand it correctly, we should move to the following.

# Allowed
mitemp_bt:
  rounding: True
  etc.

# Not allowed
sensor:
  - platform: mitemp_bt
    rounding: True
    etc.

It’s not a big deal if we don’t do this, but if we want to follow the guidelines, I think we should change this some day.

Issue Analytics

State:
Created 3 years ago
Comments:48

Top GitHub Comments

1reaction

Magalex2x14commented, Nov 12, 2020

Good news, thank you. We can make a preliminary announcement on the forum also.

1reaction

Ernst79commented, Nov 11, 2020

I propose I ask ludeeus tomorrow, and depending on the answer we decide what to do next.

I’ve updated the code to use ble_monitor right now, but we can change that tomorrow. After the answer, I will update the docs and prepare a 0.8.1 beta release, after your approval.

But for now, I’m going to bed.