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
Migrate EAFM to has entity name #98121
Conversation
Hey there @Jc2k, 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.
Is it required to set _attr_name to None? (Just want to be sure that's not the default or anything, otherwise this looks good if CI is happy).
When you set And since the device name was already the name of the entity, this doesn't change anything more. I believe the best benefit you get from this change is that if you edit your device name, your entity name changes as well |
I thought setting has_entity_name and not defining self.name was enough but it's been a while, so wanted to check. |
I think in .6 or .7 they added a log message warning that the device name is set as the entity name implicitly and that they should open an issue. Found an example: |
Lovely thanks for checking! |
Proposed change
Migrate EAFM to has entity name
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: