Skip to content
This repository was archived by the owner on Mar 12, 2024. It is now read-only.

Fixes on set color temp action and tradriDimmerTemp::setColor method - #42

Open
SenTzu01 wants to merge 1 commit into
treban:masterfrom
SenTzu01:master
Open

SenTzu01 wants to merge 1 commit into
treban:masterfrom
SenTzu01:master

Conversation

@SenTzu01

@SenTzu01 SenTzu01 commented Feb 7, 2022

Copy link
Copy Markdown

Made action logging more descriptive (Set color temp..... instead of Dimmed to....)
Removed return if current color equals requested color to fix a bug where color temp is not set under the below condition:
-Light currently is in color mode (e.g. red - #FF0000)

  • @_color = 100,
    setColor(100) is invoked
    => Light is not changed from color mode to colorTemp mode... @_color already has the value, therefore Promise.resolve() is returned with no action done.

Made action logging more descriptive (Set color temp..... instead of Dimmed to....)
Removed return if current color equals requested color to fix a bug where color temp is not set under the below condition:
-Light currently is in color mode (e.g. red - #FF0000)
- @_color = 100,
setColor(100) is invoked
=> Light is not changed from color mode to colorTemp mode... @_color already has the value, therefore Promise.resolve() is returned with no action done.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant