LavermanJJ / home-assistant-solarfocus

🏡 Solarfocus eco manager touch integration for Home Assistant
Apache License 2.0
14 stars 4 forks source link

[Feature Request]: Add Enumertion for "biomass boiler status line" (all except therminator) #57

Closed tinerop closed 1 year ago

tinerop commented 1 year ago

I noticed that all other biomass-boilers (Kessel) except therminator need different enumeration to how the status line. Modbus TCP Address 2401 numbers 0-59.

Currently this value for "biomass boiler status line" is only working if you have a "therminator".

Possible solution: Is it enough to add the numbers 0-59 to the existing numbers (200-344), because then i can add this and create a pull request...

or...an idea is when you choose at the setup between "therminator" and "all other biomass boilers" you could distinguish between the two different enumerations? but that's maybe not necessary...

referring to this: image

and here is the modbus entry: image

LavermanJJ commented 1 year ago

It'll be enough to add them as done with #49

tinerop commented 1 year ago

It'll be enough to add them as done with #49

Thank you! Perfect!

lein1013 commented 1 year ago

I think this issue could be closed #49 was merged