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

Fix issue with switch property not turning on #57

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

Conversation

zsaffir
Copy link

@zsaffir zsaffir commented Oct 15, 2022

With LZW45 off, using the setColor() method turns the actual device on but leaves the device's switch property set to off. This issue is not present when using the setLevel() method. This fix simply copies the set level functionality from the setLevel() method to the setColor() method which fixes the problem.

With LZW45 off, using the setColor() method turns the actual device on but leaves the device's switch property set to off.  This issue is not present when using the setLevel() method.  This fix simply copies the set level functionality from the setLevel() method to the setColor() method which fixes the problem.
@InovelliUSA
Copy link
Owner

I believe this makes it so that the ramp rate (parameter 3) is not used when the device is turning on. Could we keep it as Basic set but send a Basic Get to update the status?

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