heyajohnny / afvalinfo

Provides Home Assistant sensors for multiple Dutch waste collectors. The idea is to add more cities and features in the future.
GNU General Public License v3.0
102 stars 22 forks source link

Error by installing intergration #439

Closed marcel-1984 closed 5 months ago

marcel-1984 commented 5 months ago

I delete the hole Afval info in HACS en everywhere. I restart the hole system and install it back fromm HACS. Then when i instaale the intergration in Devices then i get the next error: Config-flow kon niet geladen worden: {"message":"Invalid handler specified"}

marcel-1984 commented 5 months ago

This shows up in the logbook:

Logger: homeassistant.config_entries Source: config_entries.py:2087 First occurred: 22:09:15 (9 occurrences) Last logged: 22:44:04

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

heyajohnny commented 5 months ago

Can you give me some more info? What version of Home Assistant are you using? What language is your Home Assistant set to? Did it work before?

marcel-1984 commented 5 months ago

Hey. I use 2024.1.3 11.4 and my system is in Dutch. I used it for al long time and worked perfect till last week.

Now I can't install it anymore. Now I use afvalbeheer

heyajohnny commented 5 months ago

I'm sorry, I can't seem to reproduce it in my current version when setting the language to Dutch. I assume you already followed the Upgrade procedure if you've upgraded from version 1.x.x to the current version? Maybe updating your Home Assistant to the latest version fixes things. Other than that, I have no clue what it could be...

dortatdtdx commented 5 months ago

Hey marcel-1984, I'm facing a similar problem (no upgrade). I followed the installation approach mentioned on https://github.com/heyajohnny/afvalinfo. However, I get the same error as the original poster and did not reacht the form to fill in the address details.

== LOG == 2024-06-03 10:25:29.570 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading flow for integration afvalinfo: cannot import name 'ConfigFlowResult' from 'homeassistant.config_entries' (/opt/homeassistant/lib/python3.12/site-packages/homeassistant/config_entries.py)

What I noticed that the file config_entries.py does contain ConfigFlow but does not contain a reference to ConfigFlowResult. Perhaps this could shed some light?

A fresh install of Afvalinfo from github version v2.1.0 Home Assistant - core 2024.3.3 Home Assistant - frontend 20240307.0 Languages both dutch / english tested.

heyajohnny commented 5 months ago

I've removed some type checking code that used ConfigFlowResult as a type. It seems to be sort of removed from Home Assistant, but I'm not really sure and can't find any documentation about it. Hopefully it's fixed now (in afvalinfo v2.1.1)

@marcel-1984 @dortatdtdx Can you please let me know if it fixes the issue?

marcel-1984 commented 5 months ago

@heyajohnny Thanks for your support. Now its work again. Super Nice 👍

dortatdtdx commented 5 months ago

Indeed it is working now! Thank you for your prompt response.

heyajohnny commented 5 months ago

A double confirmation that it's working again. Thnx!

It's really strange that these things just pop up at some users and other users have no problem at all. When I was editing the code I could see that the ConfigFlowResult wasn't in the config_entries anymore, but it didn't get any errors in my environment.

I'm glad it's fixed, but it's really frustrating that the Home Assistant documentation is so bad...

Thanks for reporting the issue! I'm closing it now.