Skip to content

WIP: Update icons in Sentry Error Tracking list for ignored/resolved errors

David O'Regan requested to merge oregand/gitlab:208755 into master

What does this MR do?

This MR is a MVC to address issue: #208755 (closed).

Previously, our error list only displayed unresolved errors. When !26205 (merged) is merged, ignored and resolved errors can now be viewed. But, these ignored and resolved errors still display the "ignore" and "resolve" buttons, even though those errors have already been ignored or resolved.

Proposal

Change the icons and the tooltips for ignored and resolved errors, utilizing the eye icon for undo ignore and the cancelled circle icon for unresolve. The tooltips can be updated to "Undo ignore" for the ignored errors, and "Unresolve" for the resolved errors.

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:

  • 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

Merge request reports