Skip to content

fix(user): make first/last name optional and hide empty name in profile#344

Open
hariomtiwari-work wants to merge 1 commit intoparalus:mainfrom
hariomtiwari-work:fix/user-name-optional-ui
Open

fix(user): make first/last name optional and hide empty name in profile#344
hariomtiwari-work wants to merge 1 commit intoparalus:mainfrom
hariomtiwari-work:fix/user-name-optional-ui

Conversation

@hariomtiwari-work
Copy link
Copy Markdown
Contributor

What does this PR change?

  • This PR fixes UI inconsistencies related to optional user name fields.

Changes

  • Removed required validation and asterisk (*) from First Name and Last Name fields:

    • Create User Dialog
    • New User Page
  • Fixed profile display issue:

    • Prevents rendering of undefined undefined
    • Hides Name section if both first and last name are not set

Steps to Reproduce (Before Fix)

  1. Create a user without first/last name
  2. Open user profile
  3. Observe undefined undefined displayed

After Fix

  • First Name and Last Name are correctly treated as optional
  • No misleading asterisk (*)
  • Profile does not display name if not set

Testing

  • Created user without name → no errors
  • Profile view → no name displayed
  • Created user with name → displays correctly

Does the PR depend on any other PRs or Issues? If yes, please list them.

  • issues 154

Checklist

I confirm, that I have...

  • [ .] Read and followed the contributing guide in CONTRIBUTING.md
  • [. ] Added tests for this PR
  • [. ] Formatted the code using npm run format (if applicable)
  • [ .] Updated documentation on the Paralus docs site (if applicable)
  • [. ] Updated CHANGELOG.md

Signed-off-by: hariomtiwari-work <hariom.tiwari@infracloud.io>
@hariomtiwari-work hariomtiwari-work force-pushed the fix/user-name-optional-ui branch from ebe1347 to a4462a4 Compare March 18, 2026 10:07
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.

1 participant