search

kpoppel / homeassistant-eforsyning

Home Assistant module enabling retrieval of regional heating data from eForsyning.
Apache License 2.0
36 stars 4 forks source link

[BUG] state_class is missing for the new version of home assistant #69

Open deepansteven opened 3 weeks ago

deepansteven commented 3 weeks ago

Please help me help you:

It seems that the new version of the home assistant is expecting "state_class" to be defined.

image

Describe the bug

image

To Reproduce

Open the integration in the newest version og homeassistant

Expected behavior

A clear and concise description of what you expected to happen.

Log file output

Follow the guideline in the README.md file and post the result here. Make sure to clean the information from account information.

kpoppel commented 3 weeks ago

Well these exact 5 sensors do not have a state_class, or they do, it is just set to None which is a legal value according to the developer documentation. More people have seen this on lots of other extensions: https://github.com/home-assistant/core/issues/127363

I set them to "None" beacuse I didn't think any of the "measurement", "measurement_increasing" types make sense. But I could make them "measurement" in the sense that they represent a current value without being an actual measurement.

I will give it a thought.