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

PoolNumber is overriding deprecated methods on an instance of NumberEntity #52

Open
BillyFKidney opened this issue Aug 2, 2022 · 11 comments

Comments

@BillyFKidney
Copy link

I'm receiving the following warning upon boot of my Home Assistant instance

  • Home Assistant 2022.7.7
  • Supervisor 2022.07.0
  • Operating System 8.4
  • Frontend 20220707.1 - latest
Logger: homeassistant.components.number
Source: components/number/__init__.py:220 
Integration: Number (documentation, issues) 
First occurred: 2:02:55 PM (1 occurrences) 
Last logged: 2:02:55 PM

custom_components.intellicenter.number::PoolNumber is overriding deprecated methods on an instance of NumberEntity, this is not valid and will be unsupported from Home Assistant 2022.10. Please report it to the custom component author.
@dwradcliffe
Copy link
Contributor

I have a fix for this, will push it later this week.

@smartqasa
Copy link

Getting same message.

@dwradcliffe
Copy link
Contributor

Fixed in #55

@thewolfman56
Copy link

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.

@TheRydad
Copy link

TheRydad commented Nov 14, 2022

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.

@BillyFKidney
Copy link
Author

BillyFKidney commented Nov 14, 2022 via email

@thewolfman56
Copy link

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.

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.

@thewolfman56
Copy link

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. 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.

@TheRydad
Copy link

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.

@thewolfman56
Copy link

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.

@jlvaillant
Copy link
Owner

jlvaillant commented Nov 18, 2022 via email

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

No branches or pull requests

6 participants