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

Refresh alerts list after acknowledging in flyout #125

Closed
annie3431 opened this issue Oct 4, 2021 · 0 comments
Closed

Refresh alerts list after acknowledging in flyout #125

annie3431 opened this issue Oct 4, 2021 · 0 comments
Labels
enhancement New feature or request v1.3.0

Comments

@annie3431
Copy link
Contributor

annie3431 commented Oct 4, 2021

Is your feature request related to a problem? Please describe.
After acknowledging alerts in the flyout from alerts by trigger dashboard, the table within flyout does not refresh automatically, and manual refresh will throw error because the size of get alert request is malformed. Also, would be good to have toast notification upon acknowledging the alerts to verify if it succeed or failed.

Describe the solution you'd like

  • Call getAlerts upon acknowledging alerts in flyout.
  • Separate the alert by triggers size and flyout size to 2 variables.
  • Add toast notification after acknowledging alerts in all places that allows acknowledgment.

Describe alternatives you've considered
Making the alert by triggers size a constant number, and stop using the size parameter in url.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1.3.0
Projects
None yet
Development

No branches or pull requests

3 participants