-
Notifications
You must be signed in to change notification settings - Fork 22
switching to telnet and added support for the second zone #25
Conversation
stfnhmplr
commented
Feb 25, 2019
•
edited
Loading
edited
- complete rewrite
- support speaker service (volume control with eve app) Great news! Speaker is going to be a native accessory in iOS 11 #16
- added support for second zone Zone Support #19
- switched to telnet connection to support the newer avr's Power On/Standby #22 Getting repeat Can't Connect to device error #23
- removed default input and default volume
I will happily review your code in the following days. How long has it been running at your place? |
Since a few days. Did you tried it yet? |
I just installed a test the dev branch briefly with my Denon AVR-X3400H. In the Eve app I see two volume controls and two power switches: one pair for main and one pair for 2nd zone. I do not have a 2nd zone physically connected, but control of volume works for the main zone. I do not see input switching exposed anywhere. Is there a configuration option or is this absent from the plugin? Edit Some more observations:
|
I could help with this feature after this is merged and released, but it would be nice to make the second zone optional via configuration. |
@vqv Thanks for testing. The selection of inputs will only be possible with the next version. I will adapt it to the new iOS version and removed the default input option for now. @NickChristensen Thanks. That makes sense. The second Zone is now optional. I think I will merge this PR tomorrow. |