search

thebradleysanders / Monoprice-6-Zone-Audio-Controller

An Integration for the Monoprice 6 Zone Home Audio Controller, with added sensors, sound mode, balance, treble & bass support.
12 stars 2 forks source link

Multiple amp fix #10

Closed GabrielGoldsteinAnidea closed 5 months ago

GabrielGoldsteinAnidea commented 9 months ago

With little idea of what I was doing, I loaded up the HA version and it worked well with multiple amps. So I had the idea to compare the two trees and see what changed. After learning a bit about how HA works, I finally got to the crux of the issue and got it all to work.

The only issue I have in this now, which I think is only on my machine as I can't see any differences, is that the config flow text doesn't show. If you find that yourself, please let me know what you find out.

Anyway, thanks for your help doing the heavy lifting here. I hope you'll add this to your repo for the world to enjoy.

Thanks!

CoreyJ87 commented 5 months ago

I think the config is broken because of the changes to the config_flow.py that are missing

https://github.com/CoreyJ87/Monoprice-6-Zone-Audio-Controller/commit/dd6ab2f4833096cab27bd7a378dc7dc79566eb23

I took your changes and all the current HA changes and merged them. That should fix the config part. I need to cleanup the commit and test it. I don't actually have my monoprice yet lol. Just ordered it. So I cant test it until next week.

thebradleysanders commented 5 months ago

Good work, thanks for your improvements. It's been hard for me to address this as I only have the one amp. Thanks for your help. I know you changes will be used by many!