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

Add missing functionality to Bticino switches #5289

Merged
merged 2 commits into from
Jan 8, 2023

Conversation

knovoselic
Copy link
Contributor

@knovoselic knovoselic commented Jan 7, 2023

The following functionality has been added:

  • exposed controls for Identify function (blinks built-in LED 3 times to make identifying a specific switch easier)
  • exposed led_in_dark control for 4027C type switches (turns on/off build-in LED indicator)

Kristijan Novoselić and others added 2 commits January 7, 2023 17:35
The following functionality has been added:
- exposed controls for Identify functions (blinks built-in LED 3 times
  to make identifying a specific switch easier)
- exposed led_in_dark control for 4027C type switches (turns on/off
  build-in LED indicator)
@Koenkk Koenkk merged commit 44cfe0b into Koenkk:master Jan 8, 2023
@Koenkk
Copy link
Owner

Koenkk commented Jan 8, 2023

Thanks!

@knovoselic
Copy link
Contributor Author

@Koenkk one note/question. The help text for led_in_dark for K4027C was intentionally change to

Enables the built-in LED

instead of

Enables the LED when the light is turned off, allowing to see the switch in the dark

because this is a shutter switch and there is not "light turned off" state. The option simply turns on LED forever, or turns it off. Any thoughts how to better word this, as the current help message is a bit misleading?

@Koenkk
Copy link
Owner

Koenkk commented Jan 14, 2023

Just Enables the built-in LED sounds good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants