Skip to content

fix(aci): AlertRuleTriggerAction.DoesNotExist fix#118383

Merged
saponifi3d merged 2 commits into
masterfrom
jcallender/aci/fix-alert-rule-trigger-does-not-exist
Jun 24, 2026
Merged

fix(aci): AlertRuleTriggerAction.DoesNotExist fix#118383
saponifi3d merged 2 commits into
masterfrom
jcallender/aci/fix-alert-rule-trigger-does-not-exist

Conversation

@saponifi3d

Copy link
Copy Markdown
Contributor

Description

  • Created a helper, validate_object_ids_belong that will ensure the ids are part of a query set.
  • Use helper where we do this logic already in _handler_triggers
  • Use helper to verify the alert_rule_trigger ids, if they fail lookup, then return a validation error instead. This includes a warning that the alert may be incompatible with the legacy api.

Fixes SENTRY-5KVG

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 24, 2026
@saponifi3d saponifi3d marked this pull request as ready for review June 24, 2026 19:29
@saponifi3d saponifi3d requested a review from a team as a code owner June 24, 2026 19:29
@saponifi3d saponifi3d merged commit 8d5dd78 into master Jun 24, 2026
64 checks passed
@saponifi3d saponifi3d deleted the jcallender/aci/fix-alert-rule-trigger-does-not-exist branch June 24, 2026 20:12
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