Add config flow to Brottsplatskartan#70233
Conversation
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
|
Not stale. Awaiting review. |
e7ad3ef to
ef3f930
Compare
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
|
Not stale |
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
|
Coverage 100% |
epenet
left a comment
There was a problem hiding this comment.
A few last comments, and a question.: how were multiple areas meant to work previously, and why now restrict to a single area?
It wasn't. Before and now you can only configure one area. |
|
Removed the breaking change, as this is raising an issue in the repair center. |
Breaking change
The Brottsplatskartan integration has been migrated to configuration via the UI. Configuring via YAML configuration has now been deprecated and will be removed in a future Home Assistant release.
Your existing YAML configuration is automatically imported on upgrade to this release; and thus can be safely removed from your YAML configuration after upgrading.
Proposed change
Implement config flow for brottsplatskartan
Type of change
Implementation of config flow
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: