Jayyadav1903 opened a new pull request, #4155:
URL: https://github.com/apache/streampipes/pull/4155
## Fix: User Edit Dialog Expects Username
### Problem
When editing an existing user, the User Edit Dialog enforced a required
validator on the username field.
This breaks editing for:
- self-registered users
- the initial admin user
because these accounts may not have a username.
### Solution
Relax validation **only in edit mode** for local user accounts:
- username is optional when editing
- if provided, it must still be a valid email
- creation behavior remains unchanged
### Implementation
- Updated `getUsernameValidators()` in `edit-user-dialog.component.ts`
- Removed `Validators.required` when:
- `editMode === true`
- `isUserAccount === true`
- `provider === 'local'`
### Impact
- UI-only change
- No backend or data model changes
- No breaking behavior
### Verification
- Edit local user without username → ✅ allowed
- Create local user → username still required
- Service accounts → unchanged
- External (LDAP) users → unchanged
Fixes #2681
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
