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 entity category for sensor fails mqtt sensor platform setup #103449
Conversation
Hey there @emontnemery, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
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 like this approach a lot. I assume it will still break due to the check in the sensor base platform component for other users. But at least handling mqtt sound like a very good approach.
Needs @frenck 's blessing thou.
0249f54
to
64c5764
Compare
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
We haven't introduced this for binary sensors, this accidentally came in a patch release and is reverted in #103499 For just sensors, this could help (but I am not too sure if we should ship this in a patch release tbh). |
For sensor this PR might make sense as it pops out an invalid So I could take out the binary_sensor implementation part from this PR. And split it to another PR. |
Removed the |
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.
fe1c6d2
to
f68fdf0
Compare
Let's do it 👍 |
This PR conflicts with the stable release, and cannot be cleanly pulled in for patch releasing. |
Thnx! Then well wait till the stable release. |
Proposed change
This PR introduces a grace period of 3 months to allow MQTT sensors to have an invalid entity category.
It partly reverts changes make by #103210 for binary_sensor as this is also not wanted for the patch release.
#103210 disallowed sensors and binary sensors to have an
entity_category
set toconfig
, this PR changes that behavior to an issue registration instead of failing setup.Validation for entity category was added to the sensor core component #101471 but it's correct use was already communicated at october 2021.
Related: #103511
Screenshot:
Type 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
.To help with the load of incoming pull requests: