Next Rain Forecast as None

gieljnssns / buienalarm-sensor-homeassistant

Buienalarm custom_component for Home-Assistant

MIT License

35 stars 6 forks source link

Next Rain Forecast as None #11

Closed metbril closed 3 years ago

metbril commented 3 years ago

Currently next rail forecast is shown as "-1 minutes". This doesn't look good in the UI. Why not set the sensor value to None if it is computed as -1?

If you like I can submit a PR.

metbril commented 3 years ago

Other values could be "Unavailable" or "Unknow" but just None seems most appropriate here.

gieljnssns commented 3 years ago

A PR is alway welcome…

metbril commented 3 years ago

Hope you appreciate my contributions.