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

refactor: use multi select in setup options #100

Closed xtimmy86x closed 8 months ago

xtimmy86x commented 8 months ago

Related Issues

Closes #60

Proposed Changes:

Transform the configuration of sectors in the options flow from textual input to a multi-select interface for a better and more user-friendly experience

Testing:

Extra Notes (optional):

Old method:

Screenshot 2023-11-01 143109

New method:

Screenshot 2023-10-28 151540

Checklist

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6720371468


Changes Missing Coverage Covered Lines Changed/Added Lines %
custom_components/econnect_metronet/helpers.py 9 10 90.0%
custom_components/econnect_metronet/config_flow.py 2 6 33.33%
<!-- Total: 61 66 92.42% -->
Files with Coverage Reduction New Missed Lines %
custom_components/econnect_metronet/config_flow.py 1 62.82%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 6677072536: -0.3%
Covered Lines: 480
Relevant Lines: 548

💛 - Coveralls