search

Olen / homeassistant-plant

Alternative Plant component of home assistant
308 stars 23 forks source link

Integration brakes with the release of HA 2024.7 #180

Closed corgan2222 closed 1 month ago

corgan2222 commented 3 months ago

Hello,

maybe you already know, this, but sadly the whole integration stops working after the update to HA 2024.7 which arrived today.

grafik

grafik

grafik

vg Stefan

Olen commented 3 months ago

Try the latest beta

corgan2222 commented 3 months ago

Hey Olen,

ah, I totally missed, that there was a beta version as pre-release. Thanks for pointing out. The good news is, that the plants are back and the loveace card is showing the details. So the update helped a lot.

But If I click on reload on a plant, I get this error.

grafik

If I change some config values in this plant, the details are coming back, but it still shows this error.

grafik

Let me know If I can help with testing/debugging.

vg Stefan

corgan2222 commented 3 months ago

btw, creating a new plant work without problems.

ChristophCaina commented 3 months ago

But If I click on reload on a plant, I get this error.

reloading the integration caused always this issue in the past. I think, there was already a ticket open for this.

As far as I remember, this is because the integration doesn't reload anything. It is basically just collecting the information from the source sensors...

Olen commented 3 months ago

Yeah. There is nothing to reload, so clicking that button is useless

chemelli74 commented 2 months ago

Yeah. There is nothing to reload, so clicking that button is useless

Humm, is still not handled correctly as you have a log trace and to recover you need to restart HA. Do you want a hand to trace it down ?

Side note: I changed my plant from "configure" and nothing told me that the new values where loaded in real times, so a reload was the most logic action to take.

Olen commented 2 months ago

If you feel like creating a PR that will remove the error, you are most welcome.

ChristophCaina commented 2 months ago

hm... I don't have the time / knowledge to look into this as of now (maybe, I will sit down and try to find a way to handle that at some point) - but for anyone who's interested in, this might be the area to look for improvements

https://developers.home-assistant.io/docs/config_entries_config_flow_handler/