Abdulrehman-PIAIC80387 opened a new pull request, #39942: URL: https://github.com/apache/superset/pull/39942
### SUMMARY `fetchUserOptions` in `superset-frontend/src/features/groups/utils.ts` uses `user.username` as the dropdown label. For OAuth-authenticated users (Google, etc.), the `username` is the provider's GUID rather than a human-readable name, which makes the role edit user dropdown unusable when looking up users by name. **Fix:** prefer `first_name + last_name` when available, fall back to `username` for backward compatibility. Fixes #39907 ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF **Before:** <!-- screenshot to be added --> **After:** <!-- screenshot to be added --> ### TESTING INSTRUCTIONS 1. Settings → List Users → create a user with First Name "John", Last Name "Doe", Username "108234567890123456789" (mimicking an OAuth GUID). 2. Settings → List Roles → edit any role. 3. Click in the **Users** dropdown. 4. **Before the fix:** entry shows as the OAuth GUID/username. 5. **With the fix:** entry shows as the user's full name (John Doe). Users without `first_name`/`last_name` still fall back to username. ### ADDITIONAL INFORMATION - [x] Has associated issue: Fixes #39907 - [ ] Required feature flags: - [x] Changes UI - [ ] Includes DB Migration - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
