bencevans / node-sonos

🔈 Sonos Media Player Interface/Client
https://www.npmjs.com/package/sonos
MIT License
702 stars 147 forks source link

setLEDState does not work #446

Closed hklages closed 4 years ago

hklages commented 4 years ago

I would like to setLEDState('Off') but it does not work. getLEDState works fine. Instead of 'Off' I also tried On, on, off, 0, 1, true, false - all lead to error message.

Current Behavior

I receive "Request failed with status code 500" and the stack.

Sample code or executed example

sonosPlayer.setLEDState('Off') // On | Off
      .catch(error => { console.log('Error occurred %j', error); });

Versions (and Environment)

node-sonos version: 1.12.4. Node-RED: 1.02

svrooij commented 4 years ago

You're right there was a typ-o in the DeviceProperties service. Will be fixed soon.

svrooij commented 4 years ago

Solved with https://github.com/bencevans/node-sonos/pull/447/commits/6b491292392b6a19ec3a7ac7c8c8072222c50d20

hklages commented 4 years ago

Thanks a lot. Both issues are solved (this and the playmode). Great.

svrooij commented 4 years ago

Glad to help. Check out how to sponsor me :wink: