tagyoureit / nodejs-poolController

An application to control pool equipment from various manufacturers.
GNU Affero General Public License v3.0
325 stars 94 forks source link

[BUG] homeassistant mqtt binding: missing water temp sensor discovery #665

Closed rsumner closed 1 year ago

rsumner commented 1 year ago

nodejs-poolController Version/commit

4ed27b0

nodejs-poolController-dashPanel Version/commit

No response

relayEquipmentManager Version/commit

No response

Node Version

No response

Platform

No response

RS485 Adapter

No response

Are you using Docker?

OCP

No response

Pump(s)

No response

Chlorinator(s)

No response

What steps will reproduce the bug?

Setup MQTT and HomeAssistant bindings. Look under the homeassistant->sensor-> topic and the only temperature topic that will be there is "airsensor". I have two waterSensors on my controller that have corresponding topics, but the homeassistant auto discovery topics don't get created.

What happens?

No waterSensor1 or waterSensor2 topics under home assistant auto discovery.

What should have happened?

Should have waterSensor1 and waterSensor2 topics under homeassistant

Additional information

No response

rsumner commented 1 year ago

Never mind, these temps look to be associated with the pool and spa heaters and not under the main sensors autodiscovery.