Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jonkoops
Copy link
Contributor

@jonkoops jonkoops commented 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 #24081
Closes #23790

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

cypress bot commented Oct 18, 2023

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 <[email protected]>
@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 <[email protected]>
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 <[email protected]>
@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

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