palazzem / ha-econnect-alarm

Home Assistant integration that provides a full-fledged Alarm Panel to control your Elmo/IESS alarm systems.
BSD 3-Clause "New" or "Revised" License
10 stars 5 forks source link

Changed how to handle sectors and inputs #65

Closed xtimmy86x closed 8 months ago

xtimmy86x commented 9 months ago

Related Issues

Create binary_sensor only for configured sectors #59

Proposed Changes:

I have modified how sectors and inputs binary sensors are created Now the integration create only binary sensors for sectors that are configured in control panel

Screenshoot of binary sensors with original code: after

Screenshoot of binary sensors with modified code: before

Testing:

Nothing is changed for the user configuration

Checklist

palazzem commented 8 months ago

@xtimmy86x as we discussed in Discord, we're going to refactor this part to improve the actual code and remove all the limitations we have right now.

I'm closing this PR in the meantime!