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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Konnected] Removed defaults for zone configuration parameter from the channels #13333
Conversation
Signed-off-by: Haavar Valeur <haavar@haavar.com>
@haavar - should this now be mentioned in the README? Since now required, I would assume it would need to be included in full example as bare minimum? https://next.openhab.org/addons/bindings/konnected/#full-example |
Signed-off-by: Haavar Valeur <haavar@haavar.com>
I read though the readme, and I did not see anything that would need to be updated because of this change. I did however notice that the example config used the wrong property name for the zone. I updated that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
…e channels (openhab#13333) * Removed defaults from zone parameter * Corrected the name of the zone property in example Signed-off-by: Haavar Valeur <haavar@haavar.com>
Bugfix for issue when adding channels with the default zone configured. The thing get into an invalid config state when adding a channel with the default zone selected. The only way to add channel for the default zone is to add a channel for a different zone, then go back and edit it to the default value.
The cause is that the zone parameter was marked as required and had a default.
The default value did not make sense to me, so I removed it.