davidrapan / ha-solarman

⚡ Solarman Stick Logger integration for 🏠 Home Assistant
MIT License
110 stars 25 forks source link

Make some controls disabled by default #180

Closed voed closed 1 week ago

voed commented 1 month ago

I understand this controls may be useful for someone, but i think for most people there are no any reason to change this settings from Home Assistant. But, it is pretty easy to missclick while scrolling through them which may cause damage to the battery. I suggest to make most of them disabled by default. image

davidrapan commented 1 month ago

Hi @voed, I can see where you're coming from, but the truth is that the options are pretty harmless (except for one in the SG0XXP3 profiles) and at worst it just won't work. And I say that because I've seen some BAD configurations people have had here, hah! So from this point of view, I think it is good to make those values ​​rather visible in HA.

BUT I will soon add some kind of a locking mechanism to prevent accidental changes! 😉

PV-Joe commented 1 month ago

BUT I will soon add some kind of a locking mechanism to prevent accidental changes!

Yes please. That would be great!

github-actions[bot] commented 3 weeks ago

This issue is stale because it has been open for 30 days with no activity.

github-actions[bot] commented 1 week ago

This issue was closed because it has been inactive for 14 days since being marked as stale.