Skip to content

Use danger variant on spam log remove user modal

Tom Quirk requested to merge 344168-danger-variant-spam-log-modal into master

What does this MR do and why?

For the Delete user modal:

  • Uses danger variant for modal button
  • Updates modal button text

#344168 (closed) (for epic &6854 (closed))

Screenshots or screen recordings

Before After
Screen_Shot_2022-02-15_at_2.41.16_pm Screen_Shot_2022-02-15_at_2.40.57_pm

How to set up and validate locally

  1. Enable feature flag Feature.enable(:bootstrap_confirmation_modals)
  2. Set up Akismet: https://docs.gitlab.com/ee/integration/akismet.html
  3. Navigate to the Spam logs admin tab (/admin/spam_logs)
  4. Select Remove user to reveal modal.
  5. Observe new modal.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #344168 (closed)

Merge request reports

Loading