samemory / homebridge-eufy-security

Work in progress
Apache License 2.0
97 stars 14 forks source link

[enhancement]: Option to remove "Security" mapping features. #302

Closed Haglerd closed 2 years ago

Haglerd commented 2 years ago

Describe the solution you'd like

The functionality to, "Change how each mode in the HomeKit security system is mapped to the modes in the Eufy App" would be better if you had a way to ignore it all together. I have no idea what that is or what it's asking. I don't want it to sync with anything involving my security system. An "off" or "ignore" or "do nothing" option would be nice in this instance.

Also, some better documentation around what that is and what it's actually doing would help people make the best choice given the options available.

github-actions[bot] commented 2 years ago

Did you check this Road Map pages ?

schliemann commented 2 years ago

Don't they have default values if unassigned?

schliemann commented 2 years ago

In the code they do. Don't know if the config-UI requires them.

Haglerd commented 2 years ago

They do, but that doesn’t change anything in regard to this request. It defaults to “home”.

schliemann commented 2 years ago

https://github.com/samemory/homebridge-eufy-security/blob/e86fa7983f715b7a94dbadb9fea50341bc994b9e/src/accessories/StationAccessory.ts#L144-L147

schliemann commented 2 years ago

What would you want home to default to if not "home"?

Haglerd commented 2 years ago

“ would be better if you had a way to ignore it all together. I have no idea what that is or what it's asking. I don't want it to sync with anything involving my security system. An "off" or "ignore" or "do nothing" option would be nice”

schliemann commented 2 years ago

It is one of the main functionalities of the plugin. But you can ignore stations through the config. So that is what you could try.