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): allow manual entry for LED intensity parameters for Inovelli LZW30 #7037

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anderbak
Copy link

@anderbak anderbak commented Jul 21, 2024

This pull request updates the LED intensity parameters for the Inovelli LZW30 switch.
The changes allow manual entry of values between 0 and 10 for parameters 6 and 7,
providing more granular control over LED brightness. This change aligns with
Inovelli's documentation and improves consistency across devices.

…velli LZW30 Black Series On/Off Switch with the documentation and expected behavior. By allowing manual entry for values between 0 and 10, this update ensures consistency with other Inovelli products and provides a more intuitive user experience. The new configuration allows users to set the LED intensity directly, improving the alignment with Inovelli's specifications and enhancing control.
@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Jul 21, 2024
@anderbak anderbak changed the title This change aligns the LED intensity parameters (6 and 7) with documentation and expected behavior feat(config): allow manual entry for LED intensity parameters for Inovelli LZW30 Jul 21, 2024
@AlCalzone
Copy link
Member

I'm not sure I understand the change. You can already enter values between 0 and 10 because of the predefined options.

@AlCalzone AlCalzone marked this pull request as draft August 12, 2024 13:47
@anderbak
Copy link
Author

anderbak commented Aug 31, 2024

Hello,

The predefined options are a selection of text values that correspond to the number values between 0 and 10. It is not consistent with other inovelli device configurations nor does it align with the documentation from inovelli.

LZW30 parameters pg 42
Refer to equivalent parameters 14 & 15 for LZW31

@AlCalzone AlCalzone marked this pull request as ready for review August 31, 2024 07:31
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