Skip to content

Set danger variant to remove user/project

Diana Zubova requested to merge dz/344163-set-danger-cofirmation-button-on-admin-users into master

What does this MR do and why?

Update remove confirmation when admin is removing users from groups/projects to danger variant and change the labels on remove buttons.

Related to #344163 (closed)

Screenshots or screen recordings

Without FF With FF
Remove from a project Screenshot_2022-01-27_at_12.14.24 Screenshot_2022-01-27_at_14.18.05
Remove from a group Screenshot_2022-01-27_at_12.16.24 Screenshot_2022-01-27_at_14.17.01

How to set up and validate locally

  1. Enable the bootstrap_confirmation_modals feature flag: Feature.enable(:bootstrap_confirmation_modals)
  2. Login as admin
  3. Go to http://localhost:3000/admin/users
  4. Choose a user that belongs to a group and to a project
  5. Navigate to Groups and projects tab
  6. Click a bin icon next to a group -> observe a confirmation modal
  7. Click a bin icon next to a project -> observe a confirmation 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.

Merge request reports