Danielhiversen / home_assistant_tibber_data

Display Tibber data sensors.
MIT License
39 stars 11 forks source link

State class for Monetary devices #55

Closed basbruss closed 7 months ago

basbruss commented 1 year ago

state_class=MEASUREMENT is not allowed for monetary data, according to core maintainers this should be state_class=TOTAL

Related core issue: https://github.com/home-assistant/core/issues/87243

basbruss commented 1 year ago

Set to draft until decided for correct classes in the Core repo (https://github.com/home-assistant/core/pull/87274)

tedenda commented 1 year ago

Can't we just follow how core tibber integration handle it for now? Remove state_class and release a new version. Then monitor the development in core and replicate her later. Doing this will cleanup the log for now.