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

Fix deprecation of TEMP_CELSIUS #56

Closed
wants to merge 3 commits into from
Closed

Fix deprecation of TEMP_CELSIUS #56

wants to merge 3 commits into from

Conversation

DJBenson
Copy link

@DJBenson DJBenson commented Jan 5, 2023

I made these changes to my local package and it resolved #55. Hoping this covers the entire fix.

Fix for deprecation of TEMP_CELSIUS
Fix for deprecation of TEMP_CELSIUS
Fix for deprecation of TEMP_CELSIUS
@mazzy89
Copy link

mazzy89 commented Jan 6, 2023

not needed. the fix has been already implemented 8f99381

@DJBenson
Copy link
Author

DJBenson commented Jan 6, 2023

not needed. the fix has been already implemented 8f99381

Which has completely broken my install.

@mazzy89
Copy link

mazzy89 commented Jan 6, 2023

yeah same here #57 😂 unfortunately there is not much tests coverage so we are the guinea pigs 😎

@DJBenson
Copy link
Author

DJBenson commented Jan 6, 2023

That's the fun with free/open source software 😄

@mazzy89
Copy link

mazzy89 commented Jan 6, 2023

yeah indeed.

@DJBenson
Copy link
Author

DJBenson commented Jan 6, 2023

Closing as resolved by the developer.

@DJBenson DJBenson closed this Jan 6, 2023
@zxdavb
Copy link
Owner

zxdavb commented Jan 7, 2023

Thanks for your efforts - sorry I couldn't merge your PR as was.

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

Successfully merging this pull request may close these issues.

Integration fails to load afrer upgrading to Home Assistant 2023.1.0
3 participants