-
Notifications
You must be signed in to change notification settings - Fork 15
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
PoolNumber is overriding deprecated methods on an instance of NumberEntity #52
Comments
I have a fix for this, will push it later this week. |
Getting same message. |
Fixed in #55 |
Does anyone have Home Assistant 2022.10 working with this integration? I'm still on 2022.9.7 because this is one of the most frequently used integrations for us and I cannot risk not having it available. |
I'm on 2022.10.4. I do see the unsupported warning message in my log, but everything appears to be working fine with the integration. I guess I wasn't paying attention. I am hesitant to install dwradcliffe's fork, but might look into it if it isn't merged and something breaks with another update. I don't think I have a reason to go 11 yet. |
It works on 2022.10 and 2022.11; you’ll just receive a warning. - Billy -On Nov 14, 2022, at 10:11 AM, thewolfman56 ***@***.***> wrote:
Does anyone have Home Assistant 2022.10 working with this integration? I'm still on 2022.9.7 because this is one of the most frequently used integrations for us and I cannot risk not having it available.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Thanks. I just looked into that fork and the commits look like they address this issue and may address an issue I had with circuit groups (but used scenes and switch templates in HA to work around that limitation). I may switch over though if this one starts to appear abandoned. |
Thanks. There are a couple of changes I wanted to try with 10 and 11 may end up being better, but breaking changes often has me hesitant to upgrade right away. |
The owner, @jlvaillant, had been active in here a few months ago with an issue related to 2022.06. I hope it's not abandoned. |
My concern is it looks like he has not been active in github in a few months and the pull request to correct this new error was made in September, so hoping everything is okay and the main branch can be updated. |
Yes, sorry guys, personal issues have been keeping me from being on top of things. My apologies…. I’ll try to merge things and produce a good release asap… Sent from my iPhoneOn Nov 17, 2022, at 10:53 PM, thewolfman56 ***@***.***> wrote:
The owner, @jlvaillant, had been active in here a few months ago with an issue related to 2022.06. I hope it's not abandoned.
My concern is it looks like he has not been active in github in a few months and the pull request to correct this new error was made in September, so hoping everything is okay and the main branch can be updated.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I'm receiving the following warning upon boot of my Home Assistant instance
The text was updated successfully, but these errors were encountered: