architecture icon indicating copy to clipboard operation
architecture copied to clipboard

Alarm Control Panel: Triggering services should take a cause parameter

Open elupus opened this issue 4 years ago • 1 comments

Context

Currently to keep track of what/who triggered a manual alarm, you need to keep some separate helper entity updated when triggering alarms to indicate what was the cause of the trigger. This leads to extra complexity on manual alarms.

Proposal

Allow the alarm_trigger and possibly other services to take a cause parameter that can get stored in the already existing changed_by attribute populated by some non manual alarm panels.

Consequences

The notifications based on a triggered alarm can use the attribute of the alarm panel to indicate what triggered the alarm without introducing additional helper entities.

elupus avatar Dec 27 '20 17:12 elupus

An alternative approach here would be if we can deduce the trigger cause by looking at the context parameter of the service call. But even if we support that, i think explicitly stating a cause would be nicer.

elupus avatar Dec 27 '20 17:12 elupus

This architecture issue is old, stale, and possibly obsolete. Things changed a lot over the years. Additionally, we have been moving to discussions for these architectural discussions.

For that reason, I'm going to close this issue.

../Frenck

frenck avatar May 11 '23 14:05 frenck

Still relevant, but gave up in internal stuff and moved to custom component.

elupus avatar May 11 '23 17:05 elupus