Small bug in Heater status

cryptk / haomnilogic-local

A Home Assistant integration for Hayward OmniLogic/OmniHub pool controllers using the local UDP api

Apache License 2.0

19 stars 4 forks source link

Closed mikesalz closed 3 months ago

mikesalz commented 4 months ago

I just noticed that the heater shows Off (uppercase O) or on (lowercase O). I think both should be upper, right?

cryptk commented 3 months ago

I believe that is an issue with the Home Assistant frontend. The relevant lines of code in this integration are here https://github.com/cryptk/haomnilogic-local/blob/main/custom_components/omnilogic_local/water_heater.py#L104 and I am using the STATE_ON and STATE_OFF const values (as an integration should). It's up to the frontend how it wants to render it though.

This may also be resolved by #103