Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The Alarm should not expose ack() to the TF #194

Open
acaproni opened this issue Jan 11, 2023 · 0 comments
Open

The Alarm should not expose ack() to the TF #194

acaproni opened this issue Jan 11, 2023 · 0 comments
Assignees
@acaproni
Labels
Enhancement

Comments

@acaproni
Copy link
Member

acaproni commented Jan 11, 2023 •
edited
Loading

In the implementation of the acknowledgement (#167) , the Alarm class exposes the ack() method that would allow a misbehaving TF to ack an alarm programmatically: the ACK shall only be performed in response to an operator action i.e. a command issued by the alarm GUI for example.
@acaproni acaproni changed the title The Alarm to not expose the ack() The Alarm to not expose ack() to the TF Jan 20, 2023
@acaproni acaproni changed the title The Alarm to not expose ack() to the TF The Alarm should expose ack() to the TF Jan 20, 2023
@acaproni acaproni self-assigned this Jan 20, 2023
@acaproni acaproni modified the milestone: Release 13 Jan 20, 2023
@acaproni acaproni changed the title The Alarm should expose ack() to the TF The Alarm should not expose ack() to the TF Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@acaproni acaproni

Labels
Enhancement
Projects
ias-kanban
Status: To Do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@acaproni