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

[Alerting] Improve wording on "Notify every" form input #85644

Closed
ymao1 opened this issue Dec 10, 2020 · 4 comments · Fixed by #86453
Closed

[Alerting] Improve wording on "Notify every" form input #85644

ymao1 opened this issue Dec 10, 2020 · 4 comments · Fixed by #86453
Assignees
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@ymao1
Copy link
Contributor

ymao1 commented Dec 10, 2020

This PR updated the throttle setting on the Create Alert flyout to a dropdown but kept the "Notify every" label. It was brought up that "Notify every" did not make sense with the new wording inside the dropdown.

Screen Shot 2020-12-08 at 2 45 59 PM

Discuss if we should change the label, change the dropdown wording, or wait until we move the options to the bottom of the flyout (as shown here) and update the wording at that point.

@ymao1 ymao1 added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Dec 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@arisonl
Copy link
Contributor

arisonl commented Dec 15, 2020

cc @gchaps

@ymao1 ymao1 self-assigned this Dec 16, 2020
@gchaps
Copy link
Contributor

gchaps commented Dec 17, 2020

@arisonl, @ymao1 How about tweaking both the label and the menu items as follows:

Notify

Only on status change
Actions run when the alert status changes

Every time alert is active
Actions run with every active alert interval

On a custom action interval
Actions run using the interval you set

@ymao1
Copy link
Contributor Author

ymao1 commented Dec 17, 2020

That flows much better!

@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants