Repository navigation
Allow setting default value for userprofile attribute - #39746
Conversation
|
@huyenvu2101 - thank you for the PR. Please add a sign-off to your commit like outlined in the @pedroigor - maybe your team can have a look at this PR? |
f953006 to
fa058c4
Compare
|
Just wanted to highlight that I believe this feature would be highly valuable, as it simplifies default user attribute management. |
5f8d9c2 to
ea74433
Compare
|
@pedroigor I hope you can take a look at this PR soon, we're really looking forward |
|
Hope this PR will be merged soon. I also get the same problem with the default value |
|
Maybe one of the @keycloak/core-iam team members can have a look a this PR. |
|
Just wanted to confirm that this feature is highly awaited 🤓 |
There was a problem hiding this comment.
Thanks @huyenvu2101, It looks good to me. I have requested @pedroigor to take a look and trigger the CI
There was a problem hiding this comment.
@huyenvu2101 I cannot think why one would want to set a default for, let's say, the username, the same for others. We should avoid setting a default value for root attributes (username, email, first and last names).
From a UX perspective, the Default value should probably be hidden for the root attributes.
918bbd0 to
3d0ea5f
Compare
|
@huyenvu2101 Added a commit to:
|
3d0ea5f to
4c2730d
Compare
It sounds good to me, thanks @pedroigor for updating |
Closes keycloak#36160 Signed-off-by: huyenvu2101 <vhuyen2101@gmail.com>
Signed-off-by: Pedro Igor <pigor.craveiro@gmail.com>
Signed-off-by: Pedro Igor <pigor.craveiro@gmail.com>
Signed-off-by: Pedro Igor <pigor.craveiro@gmail.com>
4c2730d to
f088e41
Compare
|
@huyenvu2101 Thanks for your time and help! |
Closes #36160
Note: Attributes with a default value will no longer be set to null, as the default value is automatically applied.