-
-
Notifications
You must be signed in to change notification settings - Fork 12
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 deprications and add unit of measurement to battery_level sensor #53
Conversation
Rebase from current HA installation
Add % to sensor state Battery_level
add german translation
c143558 - adds german translation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes correct due to deprecation:
VOLUME_MILLILITERS substituted by UnitOfVolume.MILLILITERS
MASS_GRAMS substituted by UnitOfMass.GRAMS
DEVICE_CLASS_BATTERY substituted by SensorDeviceClass.BATTERY
DEVICE_CLASS_CONNECTIVITY substituted by BinarySensorDeviceClass.CONNECTIVITY
DEVICE_CLASS_PRESENCE substituted by BinarySensorDeviceClass.PRESENCE
@benleb please create a pull request with the changes of @ChristophCaina - many thanks! 🍺 |
Don't think he is around anymore so not sure we will be getting anymore updates. |
Anyone willing to share a fork for this as it seems stale? |
I mean, I can revert other changes I've made in my fork - and provide the fork "as is" - but I cannot provide more support on the component. For the time being, you could manually put the changes into the folder of your custom_component. |
fixes deprication warnings from HA 2024.4.0 and 2024.7.0 close #14
changed version & codeowner + repository-link
…rom-HA-2024.4-and-2024.7 Fix deprication warnings from ha 2024.4 and 2024.7
Restructured to meet HACS requirements
provided by @WoBBeLnl
please ping me when you are "ready" :) 👍 |
Hi, ready for what? ^^ Are you in discord? |
README.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cant merge it with this 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol... ok.
I honestly thought, I removed that already...
Seems that it was re-added with recent changes.
Sorry
ready for merge :D For example the readme file (see the comment) seems not ready to be merged (and maybe some other files too?) :) |
delete because of merge :)
delete because of merge
delete because of merge
I'll create a new Pull-request... sorry... |
all good, np :) 👍 thanks for your work! |
at least, partially fixed issues:
#45
#51
#44
#36
#31