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

fix(config): correct config JSON syntax for Zooz ZSE70 800LR #7405

Conversation

hceuterpe
Copy link
Contributor

Updated the initial imported config file for the new ZooZ ZSE70 800LR outdoor sensor with meaningful descriptions, along with removing blank entries that failed linting.

Due to the new parameter types of this sensor, and slight variations compared to existing templates, I was only able to find a single meaningful template to include, and since it was just one, deliberately didn't include any templates.

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Nov 14, 2024
@hceuterpe hceuterpe changed the title Corrected config for Zooz ZSE70 800LR fix(config): Correct config JSON syntax for Zooz ZSE70 800LR Nov 17, 2024
@apella12
Copy link
Contributor

The description should use uppercase and lowercase normally, except for the first character which must be lowercase.
correct config JSON syntax for Zooz ZSE70 800LR

@hceuterpe hceuterpe changed the title fix(config): Correct config JSON syntax for Zooz ZSE70 800LR fix(config): correct config JSON syntax for Zooz ZSE70 800LR Nov 26, 2024
@hceuterpe
Copy link
Contributor Author

The description should use uppercase and lowercase normally, except for the first character which must be lowercase. correct config JSON syntax for Zooz ZSE70 800LR

Good grief. Thanks for pointing that out. :D

@tofurky
Copy link

tofurky commented Dec 3, 2024

tested this on my new zse70, seems to work fine, thanks.

@AlCalzone
Copy link
Member

I found #7496 first, so the work will continue there.

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.

4 participants