Skip to content

Avoid refresh when updating Alert status on Detail page

Tristan Read requested to merge tr-avoid-alert-refetch into master

What does this MR do?

Changes:

  • Ensures that the detail page always has correct information about the endedAt field.
  • Removes a (maybe?) redundant call to the graphQL endpoint.
  • Avoids a full-page loading screen on the frontend by fetching the right data in the initial call.

Fixes #218242 (closed).

Screenshots

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:

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

Merge request reports