Skip to content

fix(alert-rule): Stop creating external issue for alert rule test notifcation#58314

Merged
schew2381 merged 1 commit into
masterfrom
seiji/fix/stop-creating-external-issue-for-sample-events
Oct 18, 2023
Merged

fix(alert-rule): Stop creating external issue for alert rule test notifcation#58314
schew2381 merged 1 commit into
masterfrom
seiji/fix/stop-creating-external-issue-for-sample-events

Conversation

@schew2381

Copy link
Copy Markdown
Contributor

After you click Send Test Notification, we actually create an external issue for the sample event. However, the sample event is not surfaced through the UI anywhere, and having an external issue actually stops you from sending multiple test events for different Jira issue types such as story, bug, etc.

Screenshot 2023-10-17 at 7 51 03 PM

@schew2381 schew2381 requested review from a team and ceorourke October 18, 2023 02:53
@schew2381 schew2381 self-assigned this Oct 18, 2023
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 18, 2023
@codecov

codecov Bot commented Oct 18, 2023

Copy link
Copy Markdown

Codecov Report

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

@@           Coverage Diff           @@
##           master   #58314   +/-   ##
=======================================
  Coverage   79.06%   79.07%           
=======================================
  Files        5140     5140           
  Lines      223798   223799    +1     
  Branches    37677    37678    +1     
=======================================
+ Hits       176945   176963   +18     
+ Misses      41182    41171   -11     
+ Partials     5671     5665    -6     
Files Coverage Δ
.../rules/actions/integrations/create_ticket/utils.py 95.34% <100.00%> (+0.11%) ⬆️

... and 10 files with indirect coverage changes

@ceorourke ceorourke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice test 👏

@schew2381 schew2381 marked this pull request as ready for review October 18, 2023 17:21
@schew2381 schew2381 merged commit a5988f2 into master Oct 18, 2023
@schew2381 schew2381 deleted the seiji/fix/stop-creating-external-issue-for-sample-events branch October 18, 2023 17:22
@schew2381 schew2381 changed the title fix(alert-rule): Stop creating external issue for sample alert rule action event fix(alert-rule): Stop creating external issue for alert rule test notifcation Oct 18, 2023
schew2381 added a commit that referenced this pull request Oct 24, 2023
Accidentally added in this extra line in
#58314
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 3, 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.

2 participants