-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add driver command to update log config #116
Conversation
f9ad0d0
to
c2a8905
Compare
I will rebase once #118 is merged |
#118 is merged. 👍 |
c2a8905
to
14319c1
Compare
I'd like some TS/JS eyes on this PR. |
Do we want to bump the server version now, or wait until this is merged? Given that we can't currently get the current log config (that will come in the next |
We can do a release now of the server and have this PR go in a later release. |
Since we are on the latest version of |
Related to #115 and zwave-js/node-zwave-js#1754. This will allow us to add support to update the log configuration in the Python library and then expose a service in Home Assistant to update the log levels without having to restart the server.
The version bump is a breaking change to the add-on - updating the add-on would depend on home-assistant/core#46355 in order for
zwave_js
thermostats in Home Assistant to continue to function.Resolves #115