konnected-io / konnected-esphome

ESPHome firmware configuration and recipes for Konnected devices
Other
50 stars 31 forks source link

Add a flag to allow for inverted zones #11

Closed georgesdib closed 1 year ago

georgesdib commented 1 year ago

@heythisisnate

heythisisnate commented 1 year ago

Hi @georgesdib, thank you so much for the contribution effort! I understand that you're seeking a way to invert a binary_sensor input zone. While your approach of adding additional variables certainly works, our preferred/recommended way of doing this via the !extend keyword is described here: https://support.konnected.io/esphome-customize-zone-inputs-on-konnected-alarm-panel (see heading: Inverting a Sensor)

To keep the code clean and tidy, I'm declining to merge this into the official project at this time. Please let me know if the documented method doesn't work for you.

georgesdib commented 1 year ago

Hi @georgesdib, thank you so much for the contribution effort! I understand that you're seeking a way to invert a binary_sensor input zone. While your approach of adding additional variables certainly works, our preferred/recommended way of doing this via the !extend keyword is described here: https://support.konnected.io/esphome-customize-zone-inputs-on-konnected-alarm-panel (see heading: Inverting a Sensor)

To keep the code clean and tidy, I'm declining to merge this into the official project at this time. Please let me know if the documented method doesn't work for you.

Thank you for that, I was not aware, it works, thanks!