Entity naming improvement

JohNan / homeassistant-wellbeing

Get the status from your Electrolux devices connected to Wellbeing

MIT License

75 stars 20 forks source link

Entity naming improvement #25

Closed lazyoldbear closed 2 years ago

lazyoldbear commented 2 years ago

Right now, entities are named as {DEFAULT_NAME}_{self.entity_attr} which yields numbered outcome if several appliances are present:

sensor.wellbeing_temp
sensor.wellbeing_temp_2

This is extremely inconvenient: sensors are just useless in HA. Of course, it is possible to manually rename them all, but that is hardly a sustainable approach. The following would be a great (and not very difficult) improvement:

Utilise appliance name in entities names.
Allow configurable prefix for integration.

JohNan commented 2 years ago

@lazyoldbear I changed the name generation in the latest update. Have you tried it out? Let me know how it goes and if all is well we can close this issue.

lazyoldbear commented 2 years ago

Entity names are unchanged. Should I remove everything first?

JohNan commented 2 years ago

Unfortunately yes. I was not able to find a good migration path.

lazyoldbear commented 2 years ago

Removal from UI not possible (2022.4.3)
Removed manually from .storage/core.entity_registry
Restarted HA from the menu
All good

lazyoldbear commented 2 years ago

Works. Much, much better now. The only thing to wish is to use better auth mechanism, but I suppose that is not up to you.

Thanks a lot!

lazyoldbear commented 2 years ago

Fixed at least in 1.0.1 (possibly earlier).