Skip to content

Prevent editing closed health status

euko requested to merge 220867-prevent-editing-closed-health-status into master

What does this MR do?

  • Hides the edit button for health status on the sidebar when issue/epic is closed. (The requirement changed to 👇)
  • Disables the Edit button for health status on the sidebar when issue/epic is closed and show an informative tooltip message.

This MR completes the second leg of the two frontend tasks for Issue #220867 (closed).

Technically,

  • Uses gl-tooltip-directive over gl-tooltip for consistency and conciseness.
  • Retires GlDeprecatedDropdown and its related subcomponents in favor of the new versions.
  • Retires the bootstrap utility classes from status.vue
  • Fixes an existing UX bug (noted 👇 in the comments)

Screenshots

Demo

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

Issue #220867 (closed)

Edited by euko

Merge request reports

Loading