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

[mqtt.homeassistant] Support color temp on JSON schema lights #14839

Merged

Conversation

ccutrer
Copy link
Contributor

@ccutrer ccutrer commented Apr 17, 2023

also adds a color_mode channel if color temp is possible, so you can know how the bulb is behaving

@jlaur jlaur changed the title [mqtt.homeassistant] support color temp on JSON schema lights [mqtt.homeassistant] Support color temp on JSON schema lights Apr 21, 2023
Copy link
Contributor

@antroids antroids left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR!

@lolodomo
Copy link
Contributor

@ccutrer : did you see @antroids 's comments?

@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Jul 13, 2023
also adds a color_mode channel if color temp is possible, so you can
know how the bulb is behaving

Signed-off-by: Cody Cutrer <[email protected]>
@ccutrer ccutrer force-pushed the mqtt-homeassistant-json-light-color-temp branch from 39380cc to 134681a Compare November 13, 2023 21:17
@ccutrer
Copy link
Contributor Author

ccutrer commented Nov 13, 2023

Force pushed to rebase so tests would pass with changes to HSBType's handling of conversions. New changes were added as a separate commit.

@antroids
Copy link
Contributor

LGTM, thank you!

Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

@lolodomo lolodomo merged commit 27924d6 into openhab:main Nov 18, 2023
3 checks passed
@lolodomo lolodomo added this to the 4.1 milestone Nov 18, 2023
ccutrer added a commit to ccutrer/openhab-addons that referenced this pull request Nov 18, 2023
Due to openhab#15427 and openhab#14839 being developed independently, but merged
to main successively

Signed-off-by: Cody Cutrer <[email protected]>
jlaur pushed a commit that referenced this pull request Nov 18, 2023
Due to #15427 and #14839 being developed independently, but merged
to main successively

Signed-off-by: Cody Cutrer <[email protected]>
@ccutrer ccutrer deleted the mqtt-homeassistant-json-light-color-temp branch November 18, 2023 22:40
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Nov 26, 2023
…b#14839)

* [mqtt.homeassistant] support color temp on JSON schema lights

also adds a color_mode channel if color temp is possible, so you can
know how the bulb is behaving

* put color mode channel construction into buildChannels()

---------

Signed-off-by: Cody Cutrer <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Nov 26, 2023
Due to openhab#15427 and openhab#14839 being developed independently, but merged
to main successively

Signed-off-by: Cody Cutrer <[email protected]>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
…b#14839)

* [mqtt.homeassistant] support color temp on JSON schema lights

also adds a color_mode channel if color temp is possible, so you can
know how the bulb is behaving

* put color mode channel construction into buildChannels()

---------

Signed-off-by: Cody Cutrer <[email protected]>
Signed-off-by: Jørgen Austvik <[email protected]>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
Due to openhab#15427 and openhab#14839 being developed independently, but merged
to main successively

Signed-off-by: Cody Cutrer <[email protected]>
Signed-off-by: Jørgen Austvik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants