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

fix(log): forcing a full update is not considered an error #130

Closed palazzem closed 7 months ago

palazzem commented 7 months ago

Related Issues

Proposed Changes:

Changes debug level from ERROR to DEBUG when a full update is required.

Testing:

n/a

Extra Notes (optional):

This partially solves the issue in #121 where, in some cases, sensors can still be detected as unavailable. For further improvements, we should rewrite the AlarmCoordinator logic as we may need to handle updates differently.

Checklist

coveralls commented 7 months ago

Pull Request Test Coverage Report for Build 7745015757


Totals Coverage Status
Change from base Build 7744197143: 0.0%
Covered Lines: 597
Relevant Lines: 629

💛 - Coveralls