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

feat(config): add missing and new parameters for ZEN15 #7495

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

The00Dustin
Copy link
Contributor

Re-ordered parameters to match documentation
Added parameter 35 for firmware v2.0 (based on mis-documented parameter 25 in change log)
Updated parameter 20 for firmware v2.10
Added parameters 36, 175, and 176 for firmware v2.10
Added missing parameters 31 and 32
Added unit (W) to parameter 151
Re-labeled parameters 151 and 174 to more closely match documentation

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Dec 13, 2024
@The00Dustin
Copy link
Contributor Author

Inspired by #7494, I copied the label conditional to my branch, but I wonder if it should check hardwareVersion instead...

@hceuterpe
Copy link
Contributor

Inspired by #7494, I copied the label conditional to my branch, but I wonder if it should check hardwareVersion instead...

I don't think hardwareVersion check is necessary. Pretty sure you cannot successfully load the version 2.0+ firmware onto the older variant without bricking it :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙ Configuration issues or updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants