home-assistant / android

:iphone: Home Assistant Companion for Android
https://companion.home-assistant.io/
Apache License 2.0
2.35k stars 655 forks source link

Use the correct unit of measurement for device class volume for the hydration sensor #4846

Closed dshokouhi closed 4 days ago

dshokouhi commented 5 days ago

Summary

Entity sensor.dannys_pixel_9_pro_xl_daily_hydration (<class 'homeassistant.components.mobile_app.sensor.MobileAppSensor'>) is using native unit of measurement 'ml' which is not a valid unit for the device class ('volume') it is using; expected one of ['mL', 'gal', 'fl. oz.', 'L', 'CCF', 'ft³', 'm³']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mobile_app%22

per: https://www.home-assistant.io/integrations/sensor/

volume: Generic volume in L, mL, gal, fl. oz., m³, ft³, or CCF

oops 🙈

Screenshots

Link to pull request in Documentation repository

Documentation: home-assistant/companion.home-assistant#

Any other notes

jpelgrom commented 4 days ago

Sorry for creating this issue... (but why does core care?)