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 Shelly Wave 1, Wave 2PM, update Wave 1PM association labels #6326

Merged
merged 7 commits into from
Sep 28, 2023
Merged

feat(config): add Shelly Wave 1, Wave 2PM, update Wave 1PM association labels #6326

merged 7 commits into from
Sep 28, 2023

Conversation

QubinoHelp
Copy link
Contributor

fix: Wave 1 added zwavealliance id
feat: config new device Wave 2PM
fix: Wave 1PM association text

Igor Magajne added 4 commits September 26, 2023 12:09
Added configuration file for the Qubio Wave 1.
Associations, Parameters and Inclusion, exclusion, reser metadata.
configuration file include associations, Parameters
and metadata for inclusion, exclusion and reset
@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Sep 26, 2023
@QubinoHelp QubinoHelp changed the title Import config shelly qubino 00002 Import-config-shelly-qubino-00002 Sep 26, 2023
@QubinoHelp QubinoHelp changed the title Import-config-shelly-qubino-00002 feat: import-config-shelly-qubino-00002 Sep 26, 2023

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
@AlCalzone AlCalzone changed the title feat: import-config-shelly-qubino-00002 feat(config): add Shelly Wave 1, Wave 2PM, update Wave 1PM association labels Sep 28, 2023
Copy link
Member

@AlCalzone AlCalzone left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

FYI, I've very recently (several minutes ago) added a few custom lint rules to help with config files.
If your local development setup follows our recommendations, VSCode will highlight unnecessary properties and automatically remove them when saving the file.

@AlCalzone AlCalzone merged commit 9709e16 into zwave-js:master Sep 28, 2023
AlCalzone added a commit that referenced this pull request Sep 29, 2023
### Bugfixes
* Ignore when a node reports `Security S0/S2 CC` to have version 0 (unsupported) although it is using that CC (#6333)

### Config file changes
* Add Shelly to manufacturers (#6339)
* Add Shelly Wave 1, Wave 2PM, update Wave 1PM association labels (#6326)
* Add Sunricher SR-ZV2833PAC (#6310)

### Changes under the hood
* Added an ESLint rule to help with deciding whether a config parameter needs to be `unsigned` or not (#6325, #6338)
@QubinoHelp
Copy link
Contributor Author

Thank you I will review the recommendations you're pointing out. Will add a new device next week.

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