search

nfarina / homebridge-sonos

Sonos plugin for homebridge: https://github.com/nfarina/homebridge
155 stars 52 forks source link

Group Coordinator Behaviour #27

Closed mattnewham closed 8 years ago

mattnewham commented 8 years ago

In response to the pull request https://github.com/nfarina/homebridge-sonos/pull/15 - the discovery of devices now works fine, however I don't agree with the dynamic topology changes as described in point 3.

I should retain "dumb" control over each speaker from within home kit, rather than overall group control which has been implemented with this pull request:

If I am listening to music from within the sonos app throughout my house, I will group all devices within the app. If I then want to stop playback in one room (via HomeKit) - (lets say I have a call), turning off (stopping) playback for the room speaker now stops ALL speakers (because they have been dynamically added to a group from the Sonos topology change).

The Sonos app has much more control over what is playing etc, so I believe we should keep simple control (play/pause/volume) tied to individual speakers within the home bridge plugin.

mattnewham commented 8 years ago

Should be a comment on the pull request