search

abmantis / whirlpool-sixth-sense

Whirlpool unofficial API for 6th Sense appliances
MIT License
14 stars 12 forks source link

Appliance Unregister test for none, retry on data fetch if auth is required. #35

Closed mkmer closed 1 year ago

mkmer commented 1 year ago

Test for None before attempting to use dictionary functions. Call the callback before registering for event changes (generates new key and headers) Retry (3 times) on fetch data if auth is required.

mkmer commented 1 year ago

Still at a bit of a loss to understand why this started acting up. It was running 100% for 3 months while we waited for reviews.
Maybe a quick review on what we are doing to keep the eventsocket running is in order? Additionally, I copied this event socket code in AIOAladdinConnect which has been solid for 6+ months. Obviously I've missed something....

mkmer commented 1 year ago

Python 3.9 test now fails without from __future__ import annotations - are we going to stop running this test?

abmantis commented 1 year ago

Python 3.9 test now fails without from __future__ import annotations - are we going to stop running this test?

I've just updated the jobs. Please update from master and this should now be fixed.