ui: Settings → Users invites through a dialog, and each role is a card that says what it grants - #7692
Merged
Conversation
…d that says what it grants The invite form sat above the list as an email field and a multi-select dropdown of roles. A closed dropdown read "Owner, User" with no hint that it was a multi-choice or what a role meant, and two fixed options cost a click to open and another to close. - An "Invite people" button beside Refresh opens an "Invite a person" dialog: the email, then one card per role with a checkbox and a short description of what the role grants (Owner: full access to the application, and manages its users; User: full access to the application). A refusal is shown inside the dialog; a successful invite closes it. "Invite again" on a failed row opens the same dialog, filled in. - The Edit roles dialog uses the same role cards. - The texts that said a change takes effect "at the next sign-in or token refresh" now say "at the next sign-in". - en-US and bg-BG gain inviteOpen, inviteTitle, inviteDescription and roleDescription.Owner/User; rolesPlaceholder goes with the dropdown. - TenantUsersSettingsIT invites through the dialog and ticks the cards. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why. Settings → Users invited a person through an email field and a multi-select dropdown of roles above the list. A closed dropdown read "Owner, User", with no hint that it was a multi-choice or what a role meant. Two fixed options also cost a click to open the list and another to close it.
What changes
roleDescription.<role>; a role without a description shows its name only):Userticked by default.inviteOpen,inviteTitle,inviteDescription,roleDescription.OwnerandroleDescription.User.rolesPlaceholderis removed with the dropdown, andemptyHintis reworded.For browser tests
tenant-users-invite-open,tenant-users-invite-dialog, andtenant-users-invite-role-<role>for the invite dialog's checkboxes.tenant-users-emailandtenant-users-invite-button.tenant-users-inviteform andtenant-users-roles-trigger.label[data-role], so alabel[for='tenant-users-edit-role-<role>']no longer exists. The checkbox ids are unchanged.Verified
TenantUsersSettingsIT: 2/2 green locally (headless Chrome, Keycloak), after a quick-build install of the branch. It now opens the dialog, checks a card's description and the defaultUser, ticksOwner, and sees the dialog close. It also edits roles through the cards.mvn formatter:validateoncomponents/resources/application-coreandtests/tests-integrations, with the formatter cache wiped.Not verified: the Bulgarian wording has not been reviewed by a native speaker.
🤖 Generated with Claude Code