Skip to content
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

Refresh Configuration CC values on load if the device config Configuration info has changed #3166

Closed
raman325 opened this issue Aug 8, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@raman325
Copy link
Contributor

raman325 commented Aug 8, 2021

Is your feature request related to a problem? Please describe.
A user reported an issue here: home-assistant/core#54181 related to an out of date config file. They had updated to a version of zwave-js that had the fixed config file, but the state of the Configuration CC must have been cached from previously. We asked them to do a re-interview (really a refreshCCValues would have done it) and that resolved the problem.

Describe the solution you'd like
It seems to me like there should be a better way to handle this that doesn't require user intervention. The solution I suggested in the summary may not be the best one, but it does express the intent I am trying to get to.

Describe alternatives you've considered
Re-interview or refresh CC values will do it but requires user intervention.

Additional context
Add any other context or screenshots about the feature request here. If you want to propose an API, share your thoughts how it could be used (syntax).

@raman325 raman325 added the enhancement New feature or request label Aug 8, 2021
@AlCalzone
Copy link
Member

AlCalzone commented Aug 9, 2021

We've discussed similar things in the past. Some config file changes require a re-interview for the changes to be applied, but I'm opposed to doing that automatically. Worst case something goes wrong while the user has no chance to intervene. If someone has built an automation that controls a door lock, potentially breaking that could be really bad.

What I could imagine is some kind of status that notifies the user that re-interviewing might be necessary.

@AlCalzone
Copy link
Member

Duplicate of #2909

@AlCalzone AlCalzone marked this as a duplicate of #2909 Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants