Skip to content

Enhance user profile attribute form UX - #24083

Merged
jonkoops merged 2 commits into
keycloak:mainfrom
jonkoops:fix-scopes
Oct 18, 2023
Merged

Enhance user profile attribute form UX#24083
jonkoops merged 2 commits into
keycloak:mainfrom
jonkoops:fix-scopes

Conversation

@jonkoops

@jonkoops jonkoops commented Oct 18, 2023

Copy link
Copy Markdown
Contributor

Enhances the user experience for the create and edit form for User Profile attribute by making the following changes:

  • Prevents the scopes from being persisted if an attribute can always be shown.
  • Hides the scopes if 'Always' is selected, as the control is not interactive.
  • Sets the attribute to be editable by admins by default.

Closes #24081
Closes #23790

@jonkoops
jonkoops requested a review from pedroigor October 18, 2023 13:15
@cypress

cypress Bot commented Oct 18, 2023

Copy link
Copy Markdown

1 flaky test on run #9476 ↗︎

0 527 48 0 Flakiness 1

Details:

Merge 8d4dd94 into 8cfd9ba...
Project: Keycloak Admin UI Commit: e204e7cf7c ℹ️
Status: Passed Duration: 12:36 💡
Started: Oct 18, 2023 4:07 PM Ended: Oct 18, 2023 4:19 PM
Flakiness  cypress/e2e/realm_settings_general_tab_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Realm settings general tab tests > Test all general tab switches Test Replay Output Screenshots

Review all test suite changes for PR #24083 ↗︎

Enhances the user experience for the create and edit form for User Profile attribute by making the following changes:

- Prevents the scopes from being persisted if an attribute can always be shown.
- Hides the scopes if 'Always' is selected, as the control is not interactive.
- Sets the attribute to be editable by admins by default.

Closes keycloak#24081
Closes keycloak#23790

Co-authored-by: Pedro Igor <pigor.craveiro@gmail.com>
@jonkoops jonkoops changed the title Don't add requested scopes if set to 'always' Enhance user profile attribute form UX Oct 18, 2023
@jonkoops jonkoops self-assigned this Oct 18, 2023
@jonkoops
jonkoops marked this pull request as ready for review October 18, 2023 15:48
@jonkoops
jonkoops requested a review from a team as a code owner October 18, 2023 15:48
@jonkoops
jonkoops merged commit 4291f4b into keycloak:main Oct 18, 2023
@jonkoops
jonkoops deleted the fix-scopes branch October 18, 2023 16:24
jonkoops added a commit to jonkoops/keycloak that referenced this pull request Oct 18, 2023
Enhances the user experience for the create and edit form for User Profile attribute by making the following changes:

- Prevents the scopes from being persisted if an attribute can always be shown.
- Hides the scopes if 'Always' is selected, as the control is not interactive.
- Sets the attribute to be editable by admins by default.

Closes keycloak#24081
Closes keycloak#23790

Co-authored-by: Pedro Igor <pigor.craveiro@gmail.com>
jonkoops added a commit that referenced this pull request Oct 19, 2023
Enhances the user experience for the create and edit form for User Profile attribute by making the following changes:

- Prevents the scopes from being persisted if an attribute can always be shown.
- Hides the scopes if 'Always' is selected, as the control is not interactive.
- Sets the attribute to be editable by admins by default.

Closes #24081
Closes #23790

Co-authored-by: Pedro Igor <pigor.craveiro@gmail.com>
@stianst stianst mentioned this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scopes still persisted when required attribute is set to Always The Enabled When setting should be set to Always by default

2 participants