Skip to content

Coffee Chat suggestions/category refactor#1149

Open
ladriennel wants to merge 6 commits into
mainfrom
al/coffee-chat-refactor
Open

Coffee Chat suggestions/category refactor#1149
ladriennel wants to merge 6 commits into
mainfrom
al/coffee-chat-refactor

Conversation

@ladriennel
Copy link
Copy Markdown
Collaborator

@ladriennel ladriennel commented Apr 12, 2026

Summary

Currently, coffee chat suggestions are stored quite inefficiently where every member has a reference to all other members satisfying each category. Every semester, you had to manually update a COFFEE_CHAT_BINGO_BOARD constant in code, run the script, and open a PR. With this refactor, the collection is now modeled after the categories: there will be 16 Firestore documents for each bingo board category with an index denoting its position on the board and storing a list of all associated members.

  • Admins can upload csv of suggestions through the UI in the admin review coffee chats page. Note: this csv shape is still very constrained to what the script expects.
  • When members request their suggestions by going to the coffee chat bingo page, the API reads the 16 documents and filters them out on their own categories (meaning that the actual data doesn't exclude "self" from any categories as before).

Notion

Test Plan

  • Verify firebase (non-prod for now) has all 16 categories with right name and right corresponding members
  • Load admin coffee chats page, make sure all categories render
  • Try uploading csv with button, should completely refresh categories + members
  • Load coffee chat form page for members, check categories render, members correct, self not included
  • Ensure old coffee chat submission logic unaffected

Notes

  • Need to fix UI in admin side since formatting seems to have gotten weird from adding csv upload button
  • Need to add edit members per category in the admin ui
  • Need to populate prod database with new collection and remove old collection when pr ready

@ladriennel ladriennel requested a review from a team as a code owner April 12, 2026 21:27
@dti-github-bot
Copy link
Copy Markdown
Member

dti-github-bot commented Apr 12, 2026

[diff-counting] Significant lines: 518.

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.

2 participants