New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve and fix Airzone config flow #70474
Improve and fix Airzone config flow #70474
Conversation
- A user can add multiple config entries for the same Airzone device for those Airzone devices that support all systems listing (systemID=0) since _async_abort_entries_match is being checked with CONF_ID always set, but if the Airzone device supports systemID=0, CONF_ID won't be set. - Extend SYSTEM_ID_SCHEMA from CONFIG_SCHEMA. - invalid_system_id error refers to CONF_ID. Signed-off-by: 脕lvaro Fern谩ndez Rojas <noltari@gmail.com>
Any particular reason not to display the system id already first time? |
Yes, newer devices support setting the system id to 0, which automatically lists all connected systems. Older devices do not support this and the system id has to be specified. |
Signed-off-by: 脕lvaro Fern谩ndez Rojas <noltari@gmail.com>
Thanks @bdraco! 馃帀 |
Proposed change
Improve and fix Airzone config flow:
Type of change
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: