keycloak-ui icon indicating copy to clipboard operation
keycloak-ui copied to clipboard

User profile attribute inconsistencies with old console

Open ssilvert opened this issue 3 years ago • 4 comments

Describe the bug

@pedroigor and I discussed a few issues with the attributes in User Profile. It seems to make sense to group the bugs in this one issue and fix all at once.

username and email attributes should not have Enabled when section. username and email attributes should not have Required switch. username and email attributes should not have Permission section.

Enabled when should default to Always If you select Scopes are requested, then switch back to Always, any scopes selected should be removed.

Under Permission, if you check Admin, then User will be automatically checked and greyed out, which is confusing. Let's just make it work like the old console where each is independent.

Version

20.0.0-SNAPSHOT

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Anything else?

No response

ssilvert avatar Aug 27 '22 00:08 ssilvert

username and email attributes should not have Permission section.

Now it is possible to configure permissions to the email attribute. In a nutshell, we need to compare the admin console with the old one for differences in how both username and email can be set.

pedroigor avatar Aug 29 '22 10:08 pedroigor

@agagancarczyk Another thing that Pedro brought up is that there could be hundreds or even thousands of scopes. At the very least, we should not fetch the list of scopes until Scopes are requested is selected. This change could be done as a different issue if you want to break it up.

ssilvert avatar Aug 29 '22 13:08 ssilvert

No problem @ssilvert yes, let's break it up and create a new issue assigned to me. Thanks

agagancarczyk avatar Aug 29 '22 13:08 agagancarczyk

No problem @ssilvert yes, let's break it up and create a new issue assigned to me. Thanks

Sure, created https://github.com/keycloak/keycloak-ui/issues/3226

agagancarczyk avatar Aug 29 '22 15:08 agagancarczyk