Add `unique_id` property to entities
See original GitHub issueFirst of all, thanks for putting together this great integration!
I was trying to assign the thermostat control in the home-assistant UI to an area (the room), so that it’s grouped with any other devices in the same room. However, I get the following message when attempting to do so (by going to Configuration
->Entities
, and clicking Wiser <room name>
, the room stat):
This entity does not have a unique ID, therefore its settings cannot be managed from the UI.
Feature request: I think if e.g. the WiserRoom
ClimateDevice
exposed a unique_id
property, then this should be possible. It’s also a recommended best practice where possible. Here are some examples from other integrations:
- Tado: https://github.com/home-assistant/core/blob/c8d4cf08d99477bbb8cfb6634697af346ddba65e/homeassistant/components/tado/climate.py#L199
- Hive: https://github.com/home-assistant/core/blob/51ece97e0d2c60c94ba63b25e9207e3f2c2daca2/homeassistant/components/hive/climate.py#L63, which references https://github.com/home-assistant/core/blob/f6fbecf963d91dab9ce44cf26f219c2f53956c2f/homeassistant/components/hive/__init__.py#L185.
- link to search for others: https://github.com/home-assistant/core/search?p=1&q=unique_id+climatedevice&unscoped_q=unique_id+climatedevice
I suspect it’d be a good idea to expose the unique_id
property for all of the entities where it’s possible, but I’m not 100% sure.
Issue Analytics
- State:
- Created 3 years ago
- Comments:9 (4 by maintainers)
@m01 the config flow in v2.0 will do one of 3 things.
Hope this helps. Mark
The dev version implements unique ids as part of config flow change.