Use `GlModal` in `_chat_name.html.haml`
requested to merge 344195-replace-data-confirm-modals-with-glmodal-in-app-views-profiles-chat_names-_chat_name-html into master
What does this MR do and why?
Update the confirmation button variant to danger
and its label to Remove
on remove nickname confirmation modal.
Screenshots or screen recordings
Before FF | After FF (with updated button variant and label) |
---|---|
How to set up and validate locally
- Enable the
bootstrap_confirmation_modals
feature flag:Feature.enable(:bootstrap_confirmation_modals)
- Login as admin
- Go to any
Preferences -> Chat
- If there isn't already a chat nickname, add one. Refer steps here. Use the
ngrok
url instead oflocalhost
in the trigger url so that slack can reach your local instance. Also, you will need to setup a free slack workspace to check this integration. Once setup, run a slash command in slack to create a nickname eg.<trigger-command> help
. - Click on
Remove
button against the chat nickname. - Modal should be shown as per screenshots.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #344195 (closed)
Edited by Sharmad Nachnolkar