Add api to set specific volume levels about onkyo.js HOT 9 CLOSED

Implemented here #18 :) See https://github.com/jupe/onkyo.js/pull/18/files#diff-04c6e90faac2675aa89e2176d2eec7d8R122R125

from onkyo.js.

Error when trying getVolume()

Error: Command for 'AUDIO:STATUS' not defined
    at Promise.try (......\node_modules\onkyo.js\lib\Onkyo.js:540:15)

setVolume() is working fine

from onkyo.js.

oh, thats true. My mistake. Here ia patch for it:
#23

from onkyo.js.

actually there was typo in test which causes this bug leak.. anyway - patch is merged and should work now as expected.

from onkyo.js.

Thanks for the module. :) Just a quick note that I think the hex volume string sent may need to be upper case. At least in my case (with an Onkyo 818) volume commands that result in lowercase hex letters like "!1MVL1e" would fail.

Changing the volumeHex as follows fixed it:

const volumeHex = volume.toString(16).toUpperCase();

Thanks..

from onkyo.js.

could you open new issue, or PR? Sounds like easy to fix, but I’ve to double check if it still works with my receiver..

from onkyo.js.

I'll create new PR to fix uppercase issue, coming soon..

from onkyo.js.

https://github.com/jupe/onkyo.js/releases/tag/v0.6.1

Thanks for the module. :) Just a quick note that I think the hex volume string sent may need to be upper case. At least in my case (with an Onkyo 818) volume commands that result in lowercase hex letters like "!1MVL1e" would fail.

from onkyo.js.

Thanks for the speedy update! 👍

from onkyo.js.