Media player: sound mode structure#17
Conversation
see this Issue in the architecture fork: home-assistant/architecture#23
docs/entity_media_player.md
Outdated
| | ---- | ---- | ------- | ----------- | ||
| | sound_mode | string | None | The current sound mode of the media player | ||
| | sound_mode_list | list | None | Dynamic list of available sound modes (set by platform) | ||
| | support_select_sound_mode | boolean | False | Boolean if select sound mode command supported |
There was a problem hiding this comment.
Should this be a flag in supported_features or just decided upon whether the sound_mode_list is empty or not? The example below should also show the way how to do it (and also include the sound_mode & sound_mode_list methods, right?
There was a problem hiding this comment.
Yeah, this should not be it's own boolean.
There was a problem hiding this comment.
I removed it.
I agree it makes more sense to just look if sound_mode_list is empty or not.
|
@balloob do you have any additional comments or do you agree with this structure? |
4 participants
see this Issue in the architecture fork: home-assistant/architecture#23