nielsfaber / alarmo

Easy to use alarm system integration for Home Assistant
1.33k stars 121 forks source link

Log an warning/error when: 'alarm doesn't get enabled due to open sensors'. #986

Open VNRARA opened 3 months ago

VNRARA commented 3 months ago

I believe that in Alarmo should log a system warning/error when alarm doesn't get enabled due to open sensors that would make debugging automations a lot easier. This regardless of the GUI settings/automations.

andyb2000 commented 2 months ago

Hi, Yes I agree with this. At the moment if a sensor is in open state and you try to arm, the system silently just doesn't do anything. It would be ideal if it could log/alert/event trigger so we can catch this and report/alert the user as otherwise they may just press "ARM" and walk away without realising.

VNRARA commented 2 months ago

To add; some users may choose not or do not know how to enable the Alarmo notification for this and later end up not realising canceling an auto canceling arm-action is possible. They'd only see the alarm not turning on thinking it's a bug.

github-actions[bot] commented 1 month ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

VNRARA commented 1 month ago

Maybe this issue should be converted to a discussion?

github-actions[bot] commented 6 days ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

VNRARA commented 4 days ago

No don't close please