Skip to content

feat: bulk update roles#1596

Open
TCMeldrum wants to merge 5 commits into
developfrom
bulk-roles-assignments
Open

feat: bulk update roles#1596
TCMeldrum wants to merge 5 commits into
developfrom
bulk-roles-assignments

Conversation

@TCMeldrum

@TCMeldrum TCMeldrum commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Description

Implemented bulk update functionality for user roles.

Motivation and Context

This change is required to enable the bulk assignment and removal of roles to multiple users at once, improving the efficiency of user management tasks.

Changes

  1. Added two new methods in the UserDataSource interface (addUsersRoles and removeUsersRoles) to handle bulk operations.
  2. Implemented the two new methods in the PostgresUserDataSource and StfcUserDataSource classes.
  3. Added new mutations (addRolesToUsers and removeRolesFromUsers) in UserMutations to expose these new functionalities to the API.

How Has This Been Tested?

Fixes Jira Issue

https://jira.ess.eu//browse/

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@TCMeldrum TCMeldrum marked this pull request as ready for review June 26, 2026 10:28
@TCMeldrum TCMeldrum requested a review from a team as a code owner June 26, 2026 10:28
@TCMeldrum TCMeldrum requested review from ACLay and removed request for a team June 26, 2026 10:28
Comment thread apps/backend/src/datasources/mockups/UserDataSource.ts Outdated
Comment thread apps/e2e/cypress/e2e/roleManagement.cy.ts Outdated
Comment thread apps/frontend/src/components/user/AssignRolesView.tsx Outdated
columns={localColumns}
actions={addAction}
options={{
search: true,

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.

The search should probably be the custom search bar that is used in the people table now, though this should probably be a separate issue as it will need to be made a custom component and be reused

@ellen-wright

Copy link
Copy Markdown
Contributor

Looks good, tiny suggestion, might be good to have some spacing between the update button and the bottom of the table
image

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants