dkarv / hacs-bwt-perla

Custom HA integration for BWT Perla devices
MIT License
14 stars 1 forks source link

`day_output, month_output, year_output` are not recognised as water counters #9

Closed raldone01 closed 7 months ago

raldone01 commented 7 months ago

day_output, month_output, year_output are not recognised as water counters and can therefore not be added to the Energy dashboard. Only total_output is correctly detected as water.

device_class: water
icon: mdi:water

Seems to be missing.

dkarv commented 7 months ago

Thanks I will add the device class with the next update. What is the reason to add the others to the energy dashboard? I think total_output is the most accurate one as it doesn't reset at any point.

raldone01 commented 7 months ago

You are right about that. total_output makes the most sense.

If the class is only relevant for the energy dashboard you can just close this issue.

I am quite new to home assistant. I would still add the icon though.

The addon is working great so far. Thanks for your work.

I am curious how did you figure out the api is there documentation somewhere or did you just curl it?

dkarv commented 7 months ago

I added icons for all entities recently, and also adjusted the device class to the output entries. It is available already in the latest beta release.

There is an official documentation, somebody uploaded it here to the repo.

raldone01 commented 7 months ago

90dc0d242211dc9d52ebb1fd98769d46338c3669