Skip to content

feat(alerts): prevent creation of transaction alerts#58242

Merged
obostjancic merged 3 commits into
masterfrom
ogi/feat/prevent-transaction-alert-creation-be
Oct 18, 2023
Merged

feat(alerts): prevent creation of transaction alerts#58242
obostjancic merged 3 commits into
masterfrom
ogi/feat/prevent-transaction-alert-creation-be

Conversation

@obostjancic

@obostjancic obostjancic commented Oct 17, 2023

Copy link
Copy Markdown
Member

@obostjancic obostjancic requested a review from a team October 17, 2023 14:46
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 17, 2023

@RaduW RaduW left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good to me.

Don't see any test for this though ? (maybe it doesn't make sense)

@codecov

codecov Bot commented Oct 17, 2023

Copy link
Copy Markdown

Codecov Report

Merging #58242 (c5e6e1a) into master (28c0176) will increase coverage by 0.00%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #58242   +/-   ##
=======================================
  Coverage   79.06%   79.06%           
=======================================
  Files        5140     5140           
  Lines      223798   223809   +11     
  Branches    37677    37680    +3     
=======================================
+ Hits       176945   176960   +15     
+ Misses      41182    41177    -5     
- Partials     5671     5672    +1     
Files Coverage Δ
src/sentry/incidents/serializers/alert_rule.py 90.58% <100.00%> (+0.48%) ⬆️

... and 4 files with indirect coverage changes

@obostjancic obostjancic marked this pull request as ready for review October 18, 2023 06:22
@obostjancic obostjancic requested a review from a team as a code owner October 18, 2023 06:22
@obostjancic obostjancic merged commit 92d7c4f into master Oct 18, 2023
@obostjancic obostjancic deleted the ogi/feat/prevent-transaction-alert-creation-be branch October 18, 2023 08:42
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prevent creation of new transaction alerts

2 participants