libpurecool icon indicating copy to clipboard operation
libpurecool copied to clipboard

Published 20 hours ago • verified publisher icon etheralm

Add call to account status endpoint before attempting to login to fix authentication.

Open bfayers opened this issue 3 years ago • 7 comments

This PR is based on the information form @shenxn here: https://github.com/etheralm/libpurecool/issues/37#issuecomment-777949022

I am also validating the account status in this to avoid unnecessary API calls to Dyson.

bfayers avatar Feb 12 '21 12:02 bfayers

Will need to update the releases.rst just to ensure it passes the Hass PR requirements.

googanhiem avatar Feb 14 '21 17:02 googanhiem

Will need to update the releases.rst just to ensure it passes the Hass PR requirements.

I figured the release/update to the library would be a different PR - I can bounce the numbers to 0.6.5 in this PR if it's preferred.

bfayers avatar Feb 15 '21 16:02 bfayers

Will need to update the releases.rst just to ensure it passes the Hass PR requirements.

I figured the release/update to the library would be a different PR - I can bounce the numbers to 0.6.5 in this PR if it's preferred.

I suspect that an additional test for 'Must first check account status' should be included within https://github.com/etheralm/libpurecool/blob/master/tests/test_dyson_account.py:TestDysonAccount

homekitter avatar Feb 15 '21 16:02 homekitter

Can we get this merged and a release being published sometime soon? I think home-assistant users are waiting eagerly for this (home-assistant/core#46400). Thank you!

timn avatar Feb 27 '21 11:02 timn

Can we get this merged and a release being published sometime soon? I think home-assistant users are waiting eagerly for this (home-assistant/core#46400). Thank you!

Hopefully! I need to get the tests to pass but once I've done that it shouldn't be too difficult - however I wouldn't expect a merge here -- I'll probably end up having to publish it under another name on pypi (any ideas?) for now since I am willing to at least attempt to keep the library functional.

bfayers avatar Feb 27 '21 14:02 bfayers

@bfayers Thank you for all the great work that you've done. Is the merge blocked by the coveralls check? Have you approached @etheralm to taking over the project?

homekitter avatar Mar 04 '21 18:03 homekitter

@bfayers Thank you for all the great work that you've done. Is the merge blocked by the coveralls check?

Possibly - that said I don't think etheralm is particularly interested in merging & maintaining anymore

Have you approached @etheralm to taking over the project?

No, though I may just publish my fork to pypi and PR hass to move over. :/

bfayers avatar Mar 04 '21 19:03 bfayers