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.

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

feat(aci): Add helpers to dual write to Actions and DataConditions #82492

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ceorourke
Copy link
Member

2nd part of https://github.com/getsentry/sentry/pull/81953/files to dual write to the Action and DataCondition models when creating AlertRuleTrigger and AlertRuleTriggerAction rows.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 20, 2024
@ceorourke ceorourke requested a review from a team December 20, 2024 20:15
@@ -30,9 +30,23 @@ class Action(DefaultFieldsModel):
__repr__ = sane_repr("id", "type")

class Type(models.TextChoices):
EMAIL = "email"
SLACK = "slack"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TYTY

@iamrajjoshi iamrajjoshi requested a review from a team December 20, 2024 20:16
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

Attention: Patch coverage is 91.80328% with 5 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ry/workflow_engine/migration_helpers/alert_rule.py 90.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #82492      +/-   ##
==========================================
+ Coverage   79.40%   80.43%   +1.03%     
==========================================
  Files        7313     7314       +1     
  Lines      322544   322644     +100     
  Branches    21039    21039              
==========================================
+ Hits       256102   259516    +3414     
+ Misses      66038    62724    -3314     
  Partials      404      404              

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