search

BenPru / luxtronik

Luxtronik integration for Home Assistant
MIT License
68 stars 23 forks source link

setup_hass_services not called for a yaml-based setup #88

Closed Guzz-T closed 10 months ago

Guzz-T commented 1 year ago

With a yaml-base setup the write service is not registered. Home-Assistant response with service luxtronik2.write not found.

With the current implementation async_setup only calls setup_internal and then returns. Adding a await hass.async_add_executor_job(setup_hass_services, hass, conf) afterwards should fix this problem.

BenPru commented 1 year ago

Which version have you installed? Is it the latest?

Guzz-T commented 1 year ago

Commit https://github.com/BenPru/luxtronik/commit/2947c3c65f3392693c1af4841b91242e0d93c712 with Homeassistant core installation 2022.9.7 (Some minor additional changes had to be made to the component to make it work with the old Homeassistant version)

BenPru commented 1 year ago

In the future release the yaml based setup is going to be disabled. If you would like to use yaml config you can use Bouni/luxtronik.