Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Skip user notification if alert group is already resolved#5145

Merged
matiasb merged 1 commit into
devfrom
matiasb/skip-notification-if-resolved
Oct 9, 2024
Merged

Skip user notification if alert group is already resolved#5145
matiasb merged 1 commit into
devfrom
matiasb/skip-notification-if-resolved

Conversation

@matiasb
Copy link
Copy Markdown
Contributor

@matiasb matiasb commented Oct 9, 2024

Sometimes a task is queued and scheduled for later (or for a retry too), but the alert group is resolved by then. Skip notification in that case.

@matiasb matiasb added pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes labels Oct 9, 2024
@matiasb matiasb marked this pull request as ready for review October 9, 2024 17:54
@matiasb matiasb requested a review from a team October 9, 2024 17:54
@matiasb matiasb added this pull request to the merge queue Oct 9, 2024
Merged via the queue into dev with commit b79c5d0 Oct 9, 2024
@matiasb matiasb deleted the matiasb/skip-notification-if-resolved branch October 9, 2024 18:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants