Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Better alert copy when removing security group #305

Open
dt2patel opened this issue Jan 1, 2025 · 0 comments
Open

Better alert copy when removing security group #305

dt2patel opened this issue Jan 1, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@dt2patel
Copy link
Contributor

dt2patel commented Jan 1, 2025

Current behavior

Screenshot 2025-01-01 at 4 43 41 PM

Message: "Removing this security group may remove [User Name]'s access to certain features or data. Are you sure you want to proceed?"
Buttons: "Remove" and "Keep Group"
@dt2patel dt2patel added the bug Something isn't working label Jan 1, 2025
@dt2patel dt2patel moved this from 📋 Backlog to 📅 Planned in Digital Experience Platform Jan 1, 2025
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 15, 2025
@priyanshee1604
Improved: Better alert message when removing the Security Group (hotw…
199a575 
…ax#305)
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 15, 2025
@priyanshee1604
Improved: Better alert message when removing the Security Group (hotw…
0b3e568 
…ax#305)
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 15, 2025
@priyanshee1604
Improved: Changed the backticks to inverted commas. (hotwax#305)
805fff5
@priyanshee1604 priyanshee1604 mentioned this issue Jan 15, 2025
Merged
1 task
@priyanshee1604 priyanshee1604 moved this from 📅 Planned to 🏗 In progress in Digital Experience Platform Jan 15, 2025
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 16, 2025
@priyanshee1604
Improved: Toast message now includes the username (hotwax#305)
8aa6473
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 16, 2025
@priyanshee1604
Revert "Improved: Toast message now includes the username (hotwax#305)"
56cce9e 
This reverts commit 8aa6473.
priyanshee1604 added a commit to priyanshee1604/users that referenced this issue Jan 16, 2025
@priyanshee1604
Improved: Toast message now includes the username.(hotwax#305)
51d8f18
ymaheshwari1 added a commit that referenced this issue Jan 16, 2025
@ymaheshwari1
Merge pull request #306 from priyanshee1604/#305
0bca7a5 
Improved:  Alert message when removing the Security Group from user detail page.(#305)
@ymaheshwari1 ymaheshwari1 moved this from 🏗 In progress to ⏸️ Dev complete in Digital Experience Platform Jan 16, 2025
@priyanshee1604 priyanshee1604 removed their assignment Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
Digital Experience Platform
Status: ⏸️ Dev complete
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dt2patel @priyanshee1604