search

dlarrick / hass-kumo

Home Assistant module interfacing with Mitsubishi mini-split units
MIT License
98 stars 21 forks source link

add /#/ to login url to match web version #122

Closed thomashillyer closed 1 year ago

thomashillyer commented 1 year ago

It appears the web login for Kumo Cloud has changed: https://app.kumocloud.com/#/login

Update the login URL in kumo_cloud_setup.py from https://app.kumocloud.com/login to https://app.kumocloud.com/#/login

Fixes #109 for me

Successfully can login but the integration doesn't properly pull the IP automatically and that must be entered manually.

dlarrick commented 1 year ago

Thanks, I will take a look at this later (traveling now).

dlarrick commented 1 year ago

kumo_cloud_setup.py is legacy code that is not used by the integration at all anymore, not since switching to the pykumo library. This is why you didn't have luck adding/changing its print statements.

Since account setup eventually worked for you, I suspect the actual issue was some problem on the KumoCloud service's end.

However, if the KumoCloud server address does actually need to be changed, the place to do it is over in the pykumo library: https://github.com/dlarrick/pykumo/blob/master/pykumo/py_kumo_cloud_account.py#L40 . Ideally, that library could be changed to accept an alternate server URL, and a config option added to Kumo to allow the user to set it.

However, I myself just tested login with the URL that's over there, and it's working for me. So I'd accept PRs to make these changes, but am not going to make them myself.