Skip to content

feat: update text on groups panel #11618

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

Open
plittlewood-rpt wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat: update text on groups panel #11618

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f1e8326
feat: update text on edit user group page in rc service app to descri…
plittlewood-rpt Sep 16, 2025
5f5b6df
feat: tone down the red pill colour
plittlewood-rpt Sep 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions ...es/rc-service/src/components/users/__tests__/__snapshots__/edit-user-groups.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ exports[`UpdateUserModal should match a snapshot where there is data 1`] = `
<p
class="el-text-base el-body-text el-has-grey-text"
>
Please use the section below to manage which groups this user belongs to:
Please use the section below to manage which groups this user belongs to. Groups in highlighted red are internal groups only and should be used with caution as they grant elevated access in different applications. If you do not understand a particular group, please seek appropriate advice and do not proceed.
</p>
<div
class="el-form-layout-has-margin el-fade-in el-form-layout"
Expand Down Expand Up @@ -74,7 +74,7 @@ exports[`UpdateUserModal should match a snapshot where there is data 1`] = `
<p
class="el-text-base el-body-text el-has-grey-text"
>
Please use the section below to manage which groups this user belongs to:
Please use the section below to manage which groups this user belongs to. Groups in highlighted red are internal groups only and should be used with caution as they grant elevated access in different applications. If you do not understand a particular group, please seek appropriate advice and do not proceed.
</p>
<div
class="el-form-layout-has-margin el-fade-in el-form-layout"
Expand Down
8 changes: 6 additions & 2 deletions packages/rc-service/src/components/users/edit-user-groups.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ const updateGroupStyles = (userGroups: GroupModel[]) => {
// @ts-ignore
if (group.type !== 'internal') return
const specificElement = inputElements.find((element) => element.textContent === group.id)
specificElement?.setAttribute('style', 'background: red; color: white')
specificElement?.setAttribute('style', 'background: #D1263D; color: white')
})
}

Expand Down Expand Up @@ -134,7 +134,11 @@ export const EditUserGroups: FC<EditUserGroupsProps> = ({ refreshUsers, user, us
return (
<form onSubmit={handleSubmit(onSubmit)}>
<Subtitle>Edit User Groups</Subtitle>
<BodyText hasGreyText>Please use the section below to manage which groups this user belongs to:</BodyText>
<BodyText hasGreyText>
Please use the section below to manage which groups this user belongs to. Groups in highlighted red are internal
groups only and should be used with caution as they grant elevated access in different applications. If you do
not understand a particular group, please seek appropriate advice and do not proceed.
</BodyText>
<FormLayout hasMargin className={elFadeIn}>
<InputWrapFull>
<MultiSelectInput
Expand Down