LavermanJJ / home-assistant-solarfocus

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

Ecotop Light 23.010 Biomas boiler state not enumerated #93

Closed lein1013 closed 4 months ago

lein1013 commented 5 months ago

Describe the bug Missing enumeration of States "Solarfocus Boiler boiler status" - shows "0" instead of "Bereitschaft"

Expected behavior Numbers shall be enumerated

Desktop (please complete the following information):

Additional context Root cause is that for EcoTop Light at least for this verison the enum starts at 0

adding this to strings.json (de.json, en.json) fixes this

      "bb_status" : {
        "state" : {
          "0": "Bereitschaft",
          "1": "Zündphase",
...

fom the ecomanger-touch_Modbus pdf image

lein1013 commented 5 months ago

de are available -> only default and en are missing will be closed by https://github.com/LavermanJJ/home-assistant-solarfocus/pull/94

lein1013 commented 5 months ago

Adding the Mod bus register description (german) I just got from Solarfocus (with permission to share on github) EcoMT_Modbus-TCP_Anleitung_v11-231212_DE.pdf it includes information about registers and at which version they are available

LavermanJJ commented 4 months ago

That's great! I'll take this information to extend the integration accordingly.