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(comparison_alerts): Change alert threshold selector in metric alerts #29027

Merged
merged 14 commits into from
Oct 6, 2021

Conversation

taylangocmen
Copy link
Contributor

@taylangocmen taylangocmen commented Oct 1, 2021

This adds a step to metric alert creation to choose between a count alert and change alert. Based on the type you select the threshold value input is treated as either a number or a percentage. If you are creating a change alert you also need to select a comparison delta.

Before:
Screen Shot 2021-10-06 at 10 18 44 AM

Non-change alert creation:
Screen Shot 2021-10-06 at 10 13 19 AM

Change alert creation:
Screen Shot 2021-10-06 at 10 13 29 AM

Jira: WOR-1223
Design: Figma

@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2021

size-limit report

Path Base Size (3aa614f) Current Size Change
src/sentry/static/sentry/dist/entrypoints/app.js 52.67 KB 52.67 KB -0.01% 🔽
src/sentry/static/sentry/dist/entrypoints/sentry.css 70.9 KB 70.9 KB 0%

@taylangocmen taylangocmen requested a review from scttcper October 6, 2021 17:19
@taylangocmen taylangocmen merged commit 40e5102 into master Oct 6, 2021
@taylangocmen taylangocmen deleted the taylangocmen/change-alerts-metric-fe branch October 6, 2021 18:44
@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants