search

dotKrad / hass-fpl

FPL Component for Home Assistant
MIT License
40 stars 20 forks source link

Fix electric #51

Closed apetrycki closed 4 months ago

apetrycki commented 4 months ago

Some APIs don't have appliance information, so this adds a check for the key instead of blindly accepting that it's there. This prevents the 'electric' error in the logs and adds an info log saying it is unavailable.

GitittomeNow commented 4 months ago

Confirming no more electric error! And seems to work well after rebooting three times. Thank you so much!

GitittomeNow commented 4 months ago

I spoke too soon, the Electric error no longer shows up. I’m now seeing the following error several times. The electric error previously just came once a day just came once.

This error originated from a custom integration.

Logger: custom_components.fpl Source: custom_components/fpl/FplMainRegionApiClient.py:486 integration: FPL (documentation, issues) First occurred: 5:38:34 PM (3 occurrences) Last logged: 6:59:12 PM

apetrycki commented 4 months ago

I get that error too, but it's unrelated. It doesn't happen all the time, so it's been difficult to track down. It occurs when calling:

                response = await self.session.post(
                    URL_APPLIANCE_USAGE.format(account=account), json=JSON
                )

I'm trying to track down another one as well.