-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add monitor configuration type #141
Add monitor configuration type #141
Conversation
@@ -58,6 +58,9 @@ class EntertainmentConfigurationType(Enum): | |||
MUSIC = "music" # Channels are organized for music synchronization | |||
THREEDEESPACE = "3dspace" # Channels are organized to provide 3d spacial effects | |||
OTHER = "other" # General use case " | |||
MONITOR = ( | |||
"monitor" # Channels are organized around content from one or several monitors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you either shorten the comment or place it above the definition ?
thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great!
Shortened the comment since putting it above the definition looked a bit weird |
Thanks! |
Is there something needed from my side to get this into a new aiohue release so it can get into Home Assistant? |
I'll publish a new version within a few days. Let me quickly check if there are any other things that need fixing |
"monitor" is a new configuration type for Entertainment Configurations and can be created through the latest Hue App.
Description is from the Hue API website.
Fixes #140
Checked it by running the example vor v2 which gave the message that "monitor" was not supported, after updating the same example ran without the message