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 UI] Add visual indicator of the next request delay between rules enable/disable switch clicking. #106276

Closed
YulNaumenko opened this issue Jul 20, 2021 · 2 comments · Fixed by #107272
Assignees
Labels
Feature:Alerting/RulesManagement Issues related to the Rules Management UX loe:small Small Level of Effort Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) UX Debt UX

Comments

@YulNaumenko
Copy link
Contributor

YulNaumenko commented Jul 20, 2021

Currently we have a Switch EUI control for changing Rule enabled property in the Rules list and Details pages.
If the user interacting (clicking) fast on the switch button, too many request are sent to the server, which could cause the Kibana errors (crash sometimes):
126186318-9572985b-5a1d-4c77-9f5f-8ee1063aa749

We might change the UX by disabling the quick clicks, but with saving UI responsiveness, because in case if we will disable switch till the server respond - this is also a bad UX (at least we should provide a enough visible spinner that something happening)

Follow up on the PR

@YulNaumenko YulNaumenko added UX Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) UX Debt Feature:Alerting/RulesManagement Issues related to the Rules Management UX labels Jul 20, 2021
@elasticmachine
Copy link
Contributor

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

@YulNaumenko YulNaumenko changed the title [Alerting UI] Add visual indicator and next request delay between rules enable/disable switch clicking or use another control. [Alerting UI] Add visual indicator of the next request delay between rules enable/disable switch clicking. Jul 20, 2021
@pmuellr
Copy link
Member

pmuellr commented Jul 20, 2021

I think there's a bug in alerting here, just opened #106292

I also think that we probably do want to disable editing the alert until the disable / enable returns.

But we might want to keep this open, to help debug the underlying alerting problem, since it makes for a simple repro :-)

@mikecote mikecote added the loe:small Small Level of Effort label Jul 21, 2021
@YulNaumenko YulNaumenko self-assigned this Jul 28, 2021
@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/RulesManagement Issues related to the Rules Management UX loe:small Small Level of Effort Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) UX Debt UX
Projects
None yet
5 participants