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): parameter update for Zooz Zen16 v2.0 #6569

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

Conversation

CodeOriginal
Copy link

@CodeOriginal CodeOriginal commented Dec 27, 2023

adds parameters available to the Zooz Zen16 v2.0 from Zooz's Documentation relating to the switch inputs which are missing from the current config files.

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Dec 27, 2023
@CodeOriginal CodeOriginal changed the title Zooz Zen16 v2.0 Parameter update feat(config): Zooz Zen16 v2.0 Parameter update Dec 27, 2023
@CodeOriginal CodeOriginal changed the title feat(config): Zooz Zen16 v2.0 Parameter update feat(config): parameter update for Zooz Zen16 v2.0 Dec 27, 2023
@@ -909,6 +909,78 @@
{
"label": "Garage door (momentary mode for Z-Wave control)",
"value": 3
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a new template, since those input types are only available in revision 2 (firmware 2.0+) of the switch.

@@ -74,6 +74,13 @@
"$import": "templates/zooz_template.json#zen16_switch_type_v2",
"label": "Switch 1: Type"
},
{
"#": "25",
"$if": "firmwareVersion >= 1.2",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"$if": "firmwareVersion >= 1.2",
"$if": "firmwareVersion >= 2.0",

Same for the other new parameters.

"$if": "firmwareVersion >= 1.2",
"$import": "templates/zooz_template.json#zen16_switch_invert_v2",
"label": "Switch 1: Invert Value",
"description": "Invert the reported values for your selected switch type in Sw1 terminals (select the same value as you chose for parameter 2 to invert the values for open and closed circuit)."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We try to keep descriptions as short as possible and avoid repeating the label in other words. Only use this if necessary to add additional information, e.g.

Suggested change
"description": "Invert the reported values for your selected switch type in Sw1 terminals (select the same value as you chose for parameter 2 to invert the values for open and closed circuit)."
"description": "Select the same value as in parameter 2 to invert the values for open and closed circuit."

Same for params 26/27

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.

None yet

2 participants