Refactor setup to solve deprecation from 2023.5

mathieu-mp / homeassistant-intex-spa

Home Assistant integration for Intex Spa

MIT License

32 stars 6 forks source link

Closed FredericMa closed 1 year ago

FredericMa commented 1 year ago

I encountered the same issue as described in #47. I googled a bit and came across a similar fix of another component: https://github.com/zachcheatham/ha-omada/commit/4d9d9c2b459caa78c4a7c9ce5e0402b1026a5112

I changed it on my system and all is back up and running.

Fixes #47

johanveld commented 1 year ago

Tested the changes and can confirm it works now.

mathieu-mp commented 1 year ago

Big thanks for your work on this Pull Request ! I will check why the tests don't pass, fix and publish.

FredericMa commented 1 year ago

Right, I didn't think about the tests, sorry. I just tried to figure out what is wrong but I'm not familiar enough with python testing.