Skip to content

Implemented Alumni DB admin dashboard#1144

Open
mshi-27 wants to merge 16 commits intomainfrom
max-alumnidb-admin-dashboard
Open

Implemented Alumni DB admin dashboard#1144
mshi-27 wants to merge 16 commits intomainfrom
max-alumnidb-admin-dashboard

Conversation

@mshi-27
Copy link
Copy Markdown
Collaborator

@mshi-27 mshi-27 commented Mar 22, 2026

Summary

This pull request implements a new feature in the admin dashboard, Edit Alumni. Provides a nice UI to add alumni to the Firestore database, supporting location (compatible with the alumni map), image upload, and more.

  • New Edit Alumni page at /admin/alumni-admin-view, linked from the main admin dashboard
  • Searchable table of alumni; Add Alumni; per-row Edit / Delete;
  • Profile image upload to Storage path alumImages/{uuid} (Cornell NetID required before upload); resolves alumImages/… paths via ImagesAPI for preview.
  • Searchable dropdown backed by existing city-coordinate records; “Add …” calls geocoding to create a CityCoordinates row when needed; on save, syncs the alumni id on city-coordinate documents (remove from old pins, add to new) so the alumni map stays consistent.
  • Endpoint POST /city-coordinates/geocode supports adding new locations
image image

https://www.figma.com/design/ApnK2bbEIQqpRAyBAv7LDR/IDOL-Ashley-Working-File?node-id=758-6310&p=f&t=nrH1eQ0BEnbw3dke-0

Test Plan

  • List/Search: Confirm table loads; search matches name, email, company, location, job fields, grad year, DTI roles, subteams.
  • Add: Create an alumni with NetID, required fields, optional image upload; save; row appears.
  • Edit: Change fields including location; save; verify Firestore alumni doc and that map-related city-coordinate docs add/remove the alumni id when location changes
  • New location: Use “Add …” in the location dropdown; confirm geocoding succeeds or shows “Location not found…” when Nominatim returns nothing.
  • Delete: Confirm modal; delete; alumni removed

Notes

  • not sure how to update openapi.yaml with /alumni and /city-coordinates

@mshi-27 mshi-27 requested a review from a team as a code owner March 22, 2026 18:24
@dti-github-bot
Copy link
Copy Markdown
Member

dti-github-bot commented Mar 22, 2026

[diff-counting] Significant lines: 1486. This diff might be too big! Developer leads are invited to review the code.

@mshi-27
Copy link
Copy Markdown
Collaborator Author

mshi-27 commented Mar 23, 2026

I will fix the linter errors causing the checks to fail ASAP, been a little busy

…vide locationName in city, state, country format.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 7, 2026

CLA assistant check
All committers have signed the CLA.

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.

3 participants