Conversation
@kgoderis : could you please review this change ? |
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.
Just tried it out - works like a charm for me 👍 |
@@ -132,6 +134,8 @@ | |||
|
|||
private Map<String, String> stateMap = Collections.synchronizedMap(new HashMap<String, String>()); | |||
|
|||
private List<SonosMusicService> msuicServices; |
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.
I don't own or use any Sonos devices. But "msuicServices" looks like a typo. :-)
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.
Good catch :-)
Ah, sorry, don't know where my head is these days... I somehow mixed up Sonos with LIFX... So forget about my testing request unless you can make your bulbs play some music 🤣
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.
lol
Channel table fully updated in the documentation with the access mode added for each channel Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Typo fixed + everything squashed. |
Channel table fully updated in the documentation with the access mode added for each channel
Signed-off-by: Laurent Garnier lg.hc@free.fr