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): added Wave devices #6610

Closed

Conversation

QubinoHelp
Copy link
Contributor

added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM
added wave_template

Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM.
added wave_template
@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Jan 17, 2024
@AlCalzone
Copy link
Member

Thanks for this. I'll try to look at this when I find some time, as I'm currently on parental leave.

@apella12
Copy link
Contributor

Not the official reviewer, but it looks like the lint-zwave errors are from not using the lower case hex (i.e. 008a, not 008A, etc.) in five files.

Also three files (001x16, 0a1p16, and 0a1x16) are duplicates? (same TYPE:ID and all versions) that could be a problem. Might want to check or combine into one with all the labels

@QubinoHelp
Copy link
Contributor Author

@apella12
Thank you for the review, I will check the files and fix the issues you mentioned.

@QubinoHelp
Copy link
Contributor Author

There was mistakes, will make a new pull request. Closing this one.

@QubinoHelp QubinoHelp closed this Jan 23, 2024
QubinoHelp pushed a commit to QubinoHelp/HA-node-zwave-js that referenced this pull request Jan 23, 2024
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM
added wave_template

fixed errors from previous pull request feat(config): added Wave devices zwave-js#6610

Fixes reportd by the lint test
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

3 participants