Fix issue when setting cw/ww brightness via temperature #5976
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this implement/fix?
This fixes an issue when CW/WW lights are used via Home Assistant, after both CW and WW values have been set below 1 (which can happen with the
random
effect as that doescall.set_publish(true);
for some reason.More details in the linked issue.
Types of changes
Related issue or feature (if applicable): fixes esphome/issues#5241
Pull request in esphome-docs with documentation (if applicable): -
Test Environment
Example entry for
config.yaml
:After enabling and disabling the
random
effect once, it can be seen that there is no way (via Home Assistant) to go back to full brightness (before this PR).Checklist:
Tests have been added to verify that the new code works (under(there are currently no tests for the light component as far as I can tell)tests/
folder).If user exposed functionality or configuration variables are added/changed:
Documentation added/updated in esphome-docs.