Skip to content

Anthony/Continuation-For-PR#3917-Role-Change-Permissions-Confirmation-Modal#5340

Open
AnthonyWeathers wants to merge 19 commits into
developmentfrom
Anthony/Continuation-For-PR#3917-Role-Change-Permissions-Confirmation-Modal
Open

Anthony/Continuation-For-PR#3917-Role-Change-Permissions-Confirmation-Modal#5340
AnthonyWeathers wants to merge 19 commits into
developmentfrom
Anthony/Continuation-For-PR#3917-Role-Change-Permissions-Confirmation-Modal

Conversation

@AnthonyWeathers

@AnthonyWeathers AnthonyWeathers commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Description

Took over the work of PR#3917, adjusting it according to feedback and added additional details to keep users aware of the changes they're making while utilizing the modal.

Related PRS (if any):

There is no backend related PR needed for this frontend one.

Main changes explained:

  • Delete file A for removing unused components …
  • Updated BasicInformationTab.jsx to import and use new file that contains the modal while passing in needed values to it
  • Created RoleChangePermissionsModal.jsx to allow admin/owner users to change the role of a user, while selecting which modified permissions to keep for the user, if any of the modified ones differ from the permissions of the user's new role.
  • Updated UserProfile.jsx to pass in a authUser object that is needed for the RoleChangePermissionsModal.jsx file

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin or owner user (not sure if admin is high enough, would appreciate a check on that)
  5. go to dashboard→ Find a user (You can use Anthony Test or Anthony Test2 for testing)→ Go to this profile→ Click Manage Role & Permissions to open modal
  6. Change role of the user, and select any permissions that are listed, to keep for added/removed, then save.
  7. Verify a success toast is shown
  8. If you did not checkbox all of the listed permissions, then go to Permissions Management to check if a permission change log for the user, was created. (A change log would only occur, if there are permission changes on the role change, if you pick a role that happens to have no permissions displayed, or you check boxed all of the permissions shown, then likely no change log is created as there would be no change in the permission list of the user)
  9. verify this new feature works in [dark mode]

Screenshots or videos of changes:

Manage Role and Permissions button in Role row on User Profile page

Manage Role and Permissions button in Role row

Manage Role and Permissions Modal

Manage Role and Permissions Modal

Example of Permissions checked with Switch to Testing Role

Permissions checked for switch to Testing Role

Success toast after role change save without error

Success toast after role change save without error

Change log showing the saved changes for Test

Change log showing the saved changes for Test

Note:

…h feature and passed in auth object needed for endpoints
@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit a35e2ad
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a5a133fe91bf80008395511
😎 Deploy Preview https://deploy-preview-5340--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jul 10, 2026

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Tested the PR locally and the functionality works as expected.
  • I'm able to change role, add and delete permission. The changes are reflecting on Permission management logs as well.
  • The dark mode needs improvement overall dashboard, permission management pages as well.
Image Image Image Image Image

Comment thread src/components/UserProfile/UserProfile.jsx Outdated
Comment thread src/components/UserProfile/RoleChangePermissionsModal.jsx Outdated
Comment thread src/components/UserProfile/RoleChangePermissionsModal.jsx Outdated

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Anthony,

I tested this PR locally.

The role and permission update flow is working fine. The changes save successfully, the success toast is shown, and the feature works in dark mode.

One small issue I noticed is on the Permission Management page in dark mode: when hovering over a row, the row is not clearly visible. The hover color/contrast could be improved for better readability.

Image Image Image Image Image Image Image

@AnthonyWeathers

Copy link
Copy Markdown
Contributor Author

Hi Anthony,

I tested this PR locally.

The role and permission update flow is working fine. The changes save successfully, the success toast is shown, and the feature works in dark mode.

One small issue I noticed is on the Permission Management page in dark mode: when hovering over a row, the row is not clearly visible. The hover color/contrast could be improved for better readability.

Thanks for the tip, I modified the styling for the change log table, does it look fine now in dark mode?

@sonarqubecloud

Copy link
Copy Markdown

@nathanah nathanah left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants