- Issue created by @bvoynick
- @bvoynick opened merge request.
- Status changed to Needs review
over 1 year ago 7:08pm 30 January 2023 - Issue was unassigned.
The purpose of removeStaleAlerts() seems to be only removing alerts that are no longer being returned from the endpoint, which makes sense.
However, in practice this function removes all alerts, because the call to .includes() is on the raw endpoint data that is an array of objects. None of theme will match the UUID string being compared.
Turn on automatic refresh of alerts, and create an alert or two. Watch elements get replaced. All will get replaced at the "stale" step.
Fix the JavaScript logic to only remove alerts not present in the latest endpoint data.
Needs review
2.0
Code