KoljaWindeler / kaco

custom integration for kaco solar inverter
15 stars 8 forks source link

Fix Kaco Current Power and Energy Today sensors after 2023.02 #11

Closed rfpronk closed 1 year ago

rfpronk commented 1 year ago

After home-assistant/core/pull/85694 that was included in the 2023.02 update the Kaco integration broke partially, no longer showing Current Power and Energy Today sensors.

With this code locally it works for me again Screenshot 2023-02-03 at 12 05 40 (No it's not a sunny day here)

Fixes KoljaWindeler/kaco/issues/10

It also increases the timeout, which solves KoljaWindeler/kaco/issues/9 (it does for me). The in build webserver is just horribly slow so 10s is not enough to fetch all data (realtime.csv which you fetch often is fast)

This also includes some other cleanups that I was already using locally and which branch I extended here.

This is running on my node for a short time now and it looks fine. But I have to go now so don't have time to double check everything. Will try to do that later today.