Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning in logs: Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups #2

Open
Aerodynamite opened this issue Feb 19, 2023 · 4 comments · May be fixed by #3

Comments

@Aerodynamite
Copy link
Contributor

Hi,

I just stumbled upon this warning in the Home Assistant logs:

2023-02-13 21:40:52.468 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for ilexconnect using this method at custom_components/ilexconnect/__init__.py, line 35: hass.config_entries.async_setup_platforms(config_entry, PLATFORMS)

I very much like this integration, and the warning indicates it will stop working in the next version of Home Assistant, which would be a shame. My Python knowledge is currently limited to being able to copy-paste some stuff to add a missing sensor like I did in the PR I just created, else I would do it myself ;).

Would you be able to update the integration to cope with the new home assistant requirements?

Thanks in advance!

@bertouttier bertouttier linked a pull request Feb 26, 2023 that will close this issue
@bertouttier bertouttier linked a pull request Feb 26, 2023 that will close this issue
@bertouttier
Copy link
Owner

@Aerodynamite could you test if this change fixes the warning? Thanks

@Aerodynamite
Copy link
Contributor Author

@bertouttier Can I try this out through HACS? Or do I need to manually clone this repo in my custom_components directory and checkout the other branch?

@Aerodynamite
Copy link
Contributor Author

@bertouttier I tried it out using the method above. The error is gone, but the integration is not showing any devices or entities anymore.

image

@Y4nnickB
Copy link

@bertouttier @Aerodynamite is there any current work in progress on this integration?
I got installed a water softener today and this integration is my only hope in integrating in into HA.
Can I help with anything to maintain this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants