search

bertouttier / ilexconnect-hacs

I-LexConnect integration for HACS
MIT License
2 stars 2 forks source link

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

Open Aerodynamite opened 1 year ago

Aerodynamite commented 1 year ago

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 commented 1 year ago

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

Aerodynamite commented 1 year ago

@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 commented 1 year ago

@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 commented 2 months ago

@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?