-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dynamic fan speeds #5
Comments
I've tried updating to this version but I only get 3 fan speeds even though I know that I definitely have 5 speeds. I can see 5 speeds in the mobile app and I also had previously updated your old code to include the other 2 fan speeds (medium-low, medium-high). |
When I go to https://api.melview.net/api/unitcapabilities.aspx I get: |
Oh I haven't implemented it yet - this is more just a thought process for when I have time to. You won't get anything from the API as it needs a post request (see https://github.com/zacharyrs/ha-melview/blob/master/melview.py#L153). |
@gazelle-hub I have a potential fix for this in |
Hi, sorry I haven't had a chance to get to this. My wife is WFH at the moment so I haven't been able to find much time to be on the computer. Should be able to get to it to test in the next couple of days. |
@zacharyrs Can confirm, the 5 fan speeds from dev branch is working, changing speed is matched in the melview app/site and my super scientific, wind speed measuring paper plane on a string. |
Sorry, I've completely neglected this - I think I'll merge it over after #9. |
Closing as it's in https://github.com/zacharyrs/ha-melview/tree/dev - will do a big merge to master in #11. |
Per #4, should get whatever fan speeds are available and expose them.
Website controller defines these with:
Gets the key in call to https://api.melview.net/api/unitcapabilities.aspx, we store under
self._caps.fanstage
.Could expose using their speed names?
The text was updated successfully, but these errors were encountered: