cgarwood / homeassistant-zwave_mqtt

Limited Pre-Release of the new OZW1.6 Z-Wave component. Currently has limited platform support. Check the README for more details.
72 stars 8 forks source link

Use the node name if it's avalible to name devices #135

Closed lifeisafractal closed 4 years ago

lifeisafractal commented 4 years ago

Prefer the open-zwave node name to name the device if it is available. If the device has been named with ozw-admin this name will be used first. This allows for easier naming and management of your z-wave network independent of Home Assistant.

marcelveldt commented 4 years ago

Sorry for the late reply on this. The node name is preferred in the final implementation that is now on Home Assistant core

lifeisafractal commented 4 years ago

No problem on the reply time. I couldn't find this component in Home Assistant Core, I only see the old zwave component based on pyopenzwave. Can you point me in the right direction? Also, if this is in Core, does that mean all future development and PRs should go there?

cgarwood commented 4 years ago

We renamed the component to OpenZWave / ozw in HA core.

uSpike commented 4 years ago

@lifeisafractal the logic is here: https://github.com/home-assistant/core/blob/8cd905487e482e874a384c2e8a02070a53adfb95/homeassistant/components/ozw/entity.py#L266 and it works well for me!