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

feat(entity): add connection status binary sensor to provide feedback in case of a disconnection #151

Closed palazzem closed 7 months ago

palazzem commented 7 months ago

Related Issues

Proposed Changes:

This change adds a binary sensor as a feedback mechanism in case the device is not connected to the Cloud API. You can use it to know if the state that you see is up to date, to trigger automations, to send notifications.

Example:

image

Testing:

  1. Add the binary sensor to your dashboard.
  2. The sensor should be off in case of connection.
  3. Disconnect your main unit (device).
  4. The sensor should be triggered.

Extra Notes (optional):

This is a follow-up PR for #148. With both changes, we keep the previous state in case of disconnection to avoid triggering unwanted automations, but at the same time we provide a good feedback mechanism.

Checklist

coveralls commented 7 months ago

Pull Request Test Coverage Report for Build 8112993791

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Totals Coverage Status
Change from base Build 8112408535: 0.04%
Covered Lines: 673
Relevant Lines: 720

💛 - Coveralls
palazzem commented 7 months ago

Merging as it's possible GitHub is experiencing some issues with their runners.