search

LavermanJJ / home-assistant-solarfocus

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

Adding Biomass registers for ecotoplight #49

Closed lein1013 closed 1 year ago

lein1013 commented 1 year ago

This should cover the changes for https://github.com/LavermanJJ/home-assistant-solarfocus/issues/42

lein1013 commented 1 year ago

updated the PR based on the comments

LavermanJJ commented 1 year ago

BTW, if you fixed review comments, feel free to also resolve the conversation.

lein1013 commented 1 year ago

How do we deal with the hassfest check? for dictionary value @ data['entity']['sensor']['bo_circulation']['state']. Got {'-1': 'Locked', '0': 'Off', '1': 'On'}

LavermanJJ commented 1 year ago

I guess for the moment we need to ignore them :(