Skip to content

chore(alerts): Remove dead alerts code#115301

Merged
ceorourke merged 1 commit into
masterfrom
ceorourke/rm-IssueAlertNotificationMessage
May 11, 2026
Merged

chore(alerts): Remove dead alerts code#115301
ceorourke merged 1 commit into
masterfrom
ceorourke/rm-IssueAlertNotificationMessage

Conversation

@ceorourke
Copy link
Copy Markdown
Member

Remove unused alerts code - NotificationActionNotificationMessage is used now in place of IssueAlertNotificationMessage and MetricAlertNotificationMessage (and the "new" ones). Unblocks #115290

@ceorourke ceorourke requested review from a team as code owners May 11, 2026 18:31
@ceorourke ceorourke requested a review from a team May 11, 2026 18:31
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 11, 2026
@ceorourke ceorourke merged commit 19c9309 into master May 11, 2026
64 checks passed
@ceorourke ceorourke deleted the ceorourke/rm-IssueAlertNotificationMessage branch May 11, 2026 20:19
ceorourke added a commit that referenced this pull request May 11, 2026
…115290)

Follow up to #115150 to remove
the issue alert columns from `NotificationMessage` as they're no longer
used and it removes the FK to `RuleFireHistory` which I'll be dropping
soon.

Relies on #115301
nikkikapadia pushed a commit that referenced this pull request May 12, 2026
Remove unused alerts code - `NotificationActionNotificationMessage` is
used now in place of `IssueAlertNotificationMessage` and
`MetricAlertNotificationMessage` (and the "new" ones). Unblocks
#115290
nikkikapadia pushed a commit that referenced this pull request May 12, 2026
…115290)

Follow up to #115150 to remove
the issue alert columns from `NotificationMessage` as they're no longer
used and it removes the FK to `RuleFireHistory` which I'll be dropping
soon.

Relies on #115301
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants