Skip to content

Alert list loading & error states

Olena Horal-Koretska requested to merge 213890-alerts-loading-state into master

What does this MR do?

Adds loading and error states to for alerts

🇺🇦 Alerts section is behind feature flag: make sure to run Feature.enable(:alert_management_minimal) in rails console

When enabled, navigate to "Your project" => Operations => Alerts

Currently graphql endpoint is not merged, so for this MR it will display loading state, then error state

Successful list loading will be implemented in scope of this issue - #213880 (closed) after BE MR is merged

Screenshots

Loading

Screen_Shot_2020-04-23_at_22.08.26

Error

Screen_Shot_2020-04-24_at_19.08.34

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #213890 (closed)

Edited by Rémy Coutable

Merge request reports