Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optional userstore "Display Name" marked as mandatory #19010

Closed
janakamarasena opened this issue Jan 19, 2024 · 4 comments · Fixed by wso2/identity-apps#5282
Closed

Optional userstore "Display Name" marked as mandatory #19010

janakamarasena opened this issue Jan 19, 2024 · 4 comments · Fixed by wso2/identity-apps#5282
Assignees
Labels
Fixed/7.0.0 Severity/Critical Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Milestone

Comments

@janakamarasena
Copy link
Member

Describe the issue:
When creating a user store under "User" config there is a mandatory attribute names Display Name. When checking the carbon management console this is an optional attribute. Furthermore having this as a mandatory attribute hinders the UX as its not intuitive what this attribute is for. The old carbon console has the following info against the attribute
"This is the attribute name to display as the display name of the user"

Screenshot 2024-01-19 at 10 56 36

Expected behavior:
Display name should not be mandatory

Environment information (Please complete the following information; remove any unnecessary fields) :

  • Product Version: IS7-beta-6-snapshot

Optional Fields

Related issues:

Suggested labels:

@janakamarasena janakamarasena added ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console) Severity/Critical Type/Bug labels Jan 19, 2024
@janakamarasena janakamarasena added this to the 7.0.0-rc1 milestone Jan 19, 2024
@brionmario brionmario self-assigned this Jan 20, 2024
@brionmario brionmario moved this from Open to In Progress in Identity Server 7.0.0 Jan 20, 2024
@brionmario
Copy link
Member

DisplayNameAttribute required property is coming as true

Screenshot 2024-01-20 at 14 20 08

@brionmario
Copy link
Member

The mandatory check was removed with: wso2/identity-apps#5282

@brionmario
Copy link
Member

Additionally, the placeholders of these fields are dynamically generated and doesn't make much sense.

Ex: Display Name here is meant for the attribute used for the displayname property of a user in the userstore. But that's not intuitive in the UI.

We need to revamp the placeholders and possibly add hints:

Tracker: @brionmario Create an issue

@brionmario brionmario moved this from In Progress to Ready To Deploy in Identity Server 7.0.0 Jan 20, 2024
@NipuniBhagya
Copy link
Contributor

Since we dynamically populate this UI based on the API response, shouldn't this be fixed from the BE as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed/7.0.0 Severity/Critical Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants