rany2 / ha-open-meteo-solar-forecast

Home Assistant Open-Meteo Solar Forecast Integration
Apache License 2.0
36 stars 2 forks source link

Unable to configure on HA pre-2024.4.0b0 #7

Closed GSV3MiaC closed 2 months ago

GSV3MiaC commented 2 months ago

I get the message "Config flow could not be loaded: {"message":"Invalid handler specified"}" when I try to load this integration (running an old version of HA, under docker). Do I need to update HA to make this work or am I doing something else wrong??

rany2 commented 2 months ago

No idea, but I'm only testing in the latest HA version.

GSV3MiaC commented 2 months ago

Further info it says "Logger: homeassistant.config_entries Source: config_entries.py:2069 First occurred: 9:32:46 PM (1 occurrences) Last logged: 9:32:46 PM

Error occurred loading flow for integration open_meteo_solar_forecast: cannot import name 'ConfigFlowResult' from 'homeassistant.config_entries' (/usr/src/homeassistant/homeassistant/config_entries.py)"

I guess you use that for something and old version of HA does not have it?

I try to avoid HA updates under docker... half my integrations will break every time.

rany2 commented 2 months ago

Then it's because your HA version is too old

rany2 commented 2 months ago

https://github.com/home-assistant/core/commit/a0e558c4576cd8228ea8af9d3533438721b22d4f

GSV3MiaC commented 2 months ago

Thanks Rany, looks like I need to schedule a day for a major update then, or else hack your Py to use the older HA handler definitions.

⁣Rgds, Dave Storey​

On 8 Jun 2024, 23:06, at 23:06, Rany @.***> wrote:

https://github.com/home-assistant/core/commit/a0e558c4576cd8228ea8af9d3533438721b22d4f

-- Reply to this email directly or view it on GitHub: https://github.com/rany2/ha-open-meteo-solar-forecast/issues/7#issuecomment-2156203903 You are receiving this because you authored the thread.

Message ID: @.***>

rany2 commented 2 months ago

I've kept this issue open because I intend to fix this issue and support both older and newer HA versions but I'll need some time before I could work on it.

GSV3MiaC commented 2 months ago

That's great. I am basically using solcast still, so it isn't urgent, I was just looking for alternatives, with fewer restrictions and maybe more customization w.r.t. Horizon and shading.

⁣Rgds, Dave Storey​

On 9 Jun 2024, 11:41, at 11:41, Rany @.***> wrote:

I've kept this issue open because I intend to fix this issue and support both older and newer HA versions but I'll need some time before I could work on it.

-- Reply to this email directly or view it on GitHub: https://github.com/rany2/ha-open-meteo-solar-forecast/issues/7#issuecomment-2156437044 You are receiving this because you authored the thread.

Message ID: @.***>

GSV3MiaC commented 2 months ago

OK, the config flow no longer errors, but the sensors(and platform) are failing to set up, so I guess something else is inconsistent with older HA release. Logs, sent to Rany.

rany2 commented 2 months ago

Logs, sent to Rany.

I haven't received anything, unfortunately