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

Added switchLevel capability support. Parameter support for Leviton 4 speed controller. #241

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

csstup
Copy link

@csstup csstup commented Aug 28, 2022

  • Added support for switchLevel capability. Previous default "zwave fan switch" DTH supported it, and its needed for better Alexa support. For instance requesting the fan turn on to "25%" instead of having to know the level names. And the speed is only controllable from the Alexa app via switchLevel.
  • Added preferences/settings for leviton 4 speed fan.

Tested on Leviton 4 speed fan controller model ZW4SF.

Please let me know if I can add anything else!

@greens
Copy link
Contributor

greens commented Jan 23, 2023

We made a conscious decision to drop the switchLevel capability as it was duplicating behavior. As you mentioned, it was originally included to make integration with SmartLighting possible, but we're trying to be a bit stricter about our capability mappings these days.

@csstup
Copy link
Author

csstup commented Jan 23, 2023

Thanks for looking at my PR, Steven!

SwitchLevel was more for Alexa integration. It would be great if ST->Alexa could tighten up fan control natively with speeds instead of switchlevel.

@greens
Copy link
Contributor

greens commented Jan 23, 2023

@csstup Definitely agree with you. I'll see if I can bring it up with our voice team.

@CLAassistant
Copy link

CLAassistant commented May 31, 2023

CLA assistant check
All committers have signed the CLA.

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.

None yet

3 participants