search

Achronite / mqtt-energenie-ener314rt

MQTT interface for Energenie ENER314-RT add-on board for the Raspberry Pi, designed for use by Home Assistant.
MIT License
13 stars 5 forks source link

Correct errors in MQTT Manual setup in README.md #58

Closed genestealer closed 7 months ago

genestealer commented 8 months ago

Errors in MQTT Manual setup, such as:

  1. Reusing Identifiers in device map, will group them. So coffee maker and dimmer will be grouped as one device.
  2. Dimmer is a light, rather than a switch?
  3. via_device missing from device map
  4. software does not exist in device map
Achronite commented 8 months ago

1: I quite like having all my OOK devices grouped together as one; it was deliberate as it helps me separate the manual MQTT stuff from the automated. I guess it is down to personal preference; it's only an example after all...

2: Agree

3 & 4: Guess these are hangovers from you previous update? sw (sw_version) is in the device map BTW, https://developers.home-assistant.io/docs/device_registry_index/

genestealer commented 8 months ago

1: I quite like having all my OOK devices grouped together as one; it was deliberate as it helps me separate the manual MQTT stuff from the automated. I guess it is down to personal preference; it's only an example after all...

2: Agree

3 & 4: Guess these are hangovers from you previous update? sw (sw_version) is in the device map BTW, https://developers.home-assistant.io/docs/device_registry_index/

Ah oh, no worries, on point 1, I can revert it back to group them as you had them, but add some comments regarding HA MQTT groupings, such as grouping all the sockets of a 4-gang into one object, but keeping them separate to other sockets, etc.