Skip to content

Show user email on admin page while editing display name#387

Merged
crthpl merged 2 commits intomainfrom
worktree-vectorized-scribbling-stallman
Apr 13, 2026
Merged

Show user email on admin page while editing display name#387
crthpl merged 2 commits intomainfrom
worktree-vectorized-scribbling-stallman

Conversation

@crthpl
Copy link
Copy Markdown
Contributor

@crthpl crthpl commented Apr 13, 2026

Summary

Show user emails consistently across the admin interface and fix user selection behavior:

  1. Display emails on the main admin page, even while editing display names
  2. Display emails in the user selection dropdown when adding users to cohorts
  3. Fix hover highlighting when multiple users have the same display name

Changes

  • Restructured user display on main admin page to always show email
  • Added email display to cohort member selection dropdown and trigger button
  • Use user ID instead of display name for dropdown item values to ensure unique identification

Test plan

  • Go to /admin page as an admin
  • Start editing a user's display name - verify email is still visible
  • Go to a cohort page (/admin/cohorts/[name])
  • Click "Add Existing User" dropdown - verify emails are shown alongside display names
  • Create two users with the same display name, hover each in the dropdown - verify only the hovered one highlights

Previously, the email was hidden when the user's display name was being edited.
Restructured the layout to always display email below the display name/edit controls.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
platform Ready Ready Preview, Comment Apr 13, 2026 10:37pm

Request Review

Display email alongside display name in both the user search dropdown
and the selected user display on the cohort page.
@crthpl crthpl force-pushed the worktree-vectorized-scribbling-stallman branch from e38f0ba to d6dd570 Compare April 13, 2026 22:41
@crthpl crthpl merged commit 0163100 into main Apr 13, 2026
8 checks passed
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