-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Bump zwave-js to 11.6.0 and zwave-js-server to 1.30.0 #3148
Conversation
phew give me a bit :D |
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.
We should also include 11.6.0 (building right now). This way the regressions from 11.2.0 should not affect anyone.
|
||
### New features | ||
|
||
- Z-Wave JS Server: Add support for inclusions that are started outside of Z-Wave JS Server |
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.
Does the integration handle this yet?
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.
not yet
zwave_js/CHANGELOG.md
Outdated
|
||
- Z-Wave JS Server: Add support for inclusions that are started outside of Z-Wave JS Server | ||
- Z-Wave JS Server: Add support for lastSeen property | ||
- Z-Wave JS: Improve support for "identify" commands |
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.
Likewise, has support for this landed in HA already?
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.
not yet
Co-authored-by: AlCalzone <[email protected]>
Co-authored-by: AlCalzone <[email protected]>
addon came up clean on my end (tested with a controller without any attached devices) |
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.
Looks good to me 👍
Feel free to merge when ready to ship.
../Frenck
…#3148) * Bump zwave-js to 11.5.3 and zwave-js-server to 1.30.0 * Update zwave_js/CHANGELOG.md Co-authored-by: AlCalzone <[email protected]> * Update zwave_js/CHANGELOG.md Co-authored-by: AlCalzone <[email protected]> * Bump zwave-js to 11.6.0 * Bump alpine to 3.18 * Revert switch to using Alpine edge repo * Remove unsafe-perm option --------- Co-authored-by: AlCalzone <[email protected]>
@AlCalzone anything else you want to highlight in the detailed changelog?