Skip to content

feat(mute-metric-alerts): Add mute button to metric alerts page #50967

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

Merged
merged 9 commits into from
Jun 15, 2023

Conversation

roggenkemper
Copy link
Member

this pr adds in the mute button and banner showing mute status to the metric alerts page.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 14, 2023
@roggenkemper roggenkemper marked this pull request as ready for review June 14, 2023 22:16
@roggenkemper roggenkemper requested a review from a team as a code owner June 14, 2023 22:16
@roggenkemper roggenkemper requested a review from a team June 14, 2023 22:16
Comment on lines 4 to 5
import type {IssueAlertRule} from 'sentry/types/alerts';
import {RuleActionsCategories} from 'sentry/types/alerts';
Copy link
Member

Choose a reason for hiding this comment

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

just combine these

Comment on lines +187 to +193
: tct(
"[creator] muted this alert[forEveryone]so you won't get these notifications in the future.",
{
creator: rule.snoozeCreatedBy,
forEveryone: rule.snoozeForEveryone ? ' for everyone ' : ' ',
}
)}
Copy link
Member

Choose a reason for hiding this comment

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

dumb question but wouldn't metric alerts always be for everyone or are we looking for specific email my team -> this user is muted

Copy link
Member

Choose a reason for hiding this comment

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

Screenshot 2023-06-15 at 12 31 25 PM

you can set it to email a user or yeah if it's a team it emails each user in the team and one person can mute it for themselves.

@roggenkemper roggenkemper merged commit eb1951e into master Jun 15, 2023
@roggenkemper roggenkemper deleted the roggenkemper/metricmutebutton branch June 15, 2023 21:25
@github-actions github-actions bot locked and limited conversation to collaborators Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants