Remove STATE_CLASS_TOTAL and last_reset from sensor#54755
Remove STATE_CLASS_TOTAL and last_reset from sensor#54755emontnemery merged 14 commits intohome-assistant:devfrom
Conversation
|
Regarding Mill: |
|
@Danielhiversen That is solved by #54751 |
|
Ahh, then the changes to the Mill integration looks good 👍 |
|
Okay with #54523 this was added and there was also some reasoning added to the PR about the idea behind it. Now one of the new state classes is removed. I think this is because total is equal to measurement and obsolete. But can not follow it fully. Where are such changes discussed and documented? Searched PR messages here, the discussion's on the architecture repo and discord. Try to understand and follow the changes (my way of getting into HA development and see where I can contribute 😄 ) |
f8c5d3f to
fa82039
Compare
| "Assistant Core 2021.11. Please update your configuration if " | ||
| "state_class is manually configured, otherwise %s", |
There was a problem hiding this comment.
| "Assistant Core 2021.11. Please update your configuration if " | |
| "state_class is manually configured, otherwise %s", | |
| "Assistant Core 2021.11. Please update your configuration and set state class to 'total_increasing' if " | |
| "state_class is manually configured, otherwise %s", |
9bcb45f to
e3b6007
Compare
last_reset attribute and set state class to total_increasing for spider energy sensors
#54822
Proposed change
Simplify sensor:
last_resetattribute from sensorType of change
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: