search

mhendriks / P1-Dongel-ESP32

P1-Dongel-ESP32
MIT License
27 stars 13 forks source link

Home assisant auto discovery MQTT topic incorrect #33

Closed Runesmacher closed 5 months ago

Runesmacher commented 5 months ago

Hi

The Topic for the water sensor in Home assistant with MQTT seems to be incorrect. Dongle v4.8.14.

The topic should be changed to water_delivered in the auto discovery

image image

mhendriks commented 5 months ago

@Runesmacher I checked it with my current version 4.8.14 b2807 ... looks okay.

Screenshot 2024-02-04 at 12 25 37
Runesmacher commented 5 months ago

I have a smart water, gas and electricity meter that all feed trough the Electricity meter, i think that is where the issue stems from. MQTT send different topics then what HASS is sent as autodiscover:

https://github.com/mhendriks/P1-Dongel-ESP32/blob/7303b3bf7d601df998bc7440a684b0d6187a8c2a/_Water.ino#L8-L17

https://github.com/mhendriks/P1-Dongel-ESP32/blob/7303b3bf7d601df998bc7440a684b0d6187a8c2a/MQTT.ino#L78

mhendriks commented 5 months ago

Ah yes this is the problem, sloppy error :-( Thanks for noticing me! It is changed to the water topic in version 4.8.15 ... now available for update :-)

Screenshot 2024-02-04 at 21 41 44
Runesmacher commented 5 months ago

Can confirm it is fixed with 4.8.15.

Thanks!