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

publishState() does not publish current brightness state #4

Closed
agc93 opened this issue Sep 25, 2018 · 0 comments · Fixed by #5
Closed

publishState() does not publish current brightness state #4

agc93 opened this issue Sep 25, 2018 · 0 comments · Fixed by #5

Comments

@agc93
Copy link
Contributor

agc93 commented Sep 25, 2018

Thanks for an awesome project!

I'm not sure if this is intended behaviour or not, but while changes in color value and power state are correctly published in the MQTT payload created in publishState(), it looks like brightness is not.

It seems like it should be possible to add the value of lightRed to a brightness key around line 466 and publish it with the rest of the state. Sorry if that's not right, I don't really know C++ very well 😐

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 a pull request may close this issue.

1 participant