Skip to content

Commit

Permalink
[Serverless] Update custom role deletion modal text to reflect multip…
Browse files Browse the repository at this point in the history
…le roles (#184529)

Closes #184116

## Summary

Update the text in the role deletion confirmation modal in Serverless to
clearly indicate possible consequences for users when multiple roles are
being deleted.
  • Loading branch information
SiddharthMantri authored May 31, 2024
1 parent b5ef72b commit 5e8d3e8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ export class DeleteRoleButton extends Component<Props, State> {
) : (
<FormattedMessage
id="xpack.security.management.roles.confirmDelete.serverless.removingSingleRoleDescription"
defaultMessage="Users with the {roleName} role assigned will lose access to the project."
defaultMessage="Users with the {roleName} role assigned may lose access to the project if they are not assigned any other roles."
values={{ roleName }}
/>
)}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ export class ConfirmDelete extends Component<Props, State> {
<p>
<FormattedMessage
id="xpack.security.management.roles.confirmDelete.serverless.removingRolesDescription"
defaultMessage="Users with the following roles assigned will lose access to the project:"
defaultMessage="Users with the following roles assigned may lose access to the project if they are not assigned any other roles:"
/>
</p>
)}
Expand All @@ -99,7 +99,7 @@ export class ConfirmDelete extends Component<Props, State> {
<p>
<FormattedMessage
id="xpack.security.management.roles.confirmDelete.serverless.removingSingleRoleDescription"
defaultMessage="Users with the {roleName} role assigned will lose access to the project."
defaultMessage="Users with the {roleName} role assigned may lose access to the project if they are not assigned any other roles."
values={{ roleName: rolesToDelete[0] }}
/>
</p>
Expand Down

0 comments on commit 5e8d3e8

Please sign in to comment.