search

cornell-dti / idol

Internal tools platform for DTI, a project team/web development club at Cornell University, which automates all operations of an organization of 100+ members.
GNU Affero General Public License v3.0
7 stars 2 forks source link

IDOL: Add Semester Joined and College to User Profile #733

Closed cchrischen closed 1 week ago

cchrischen commented 1 week ago

Summary

This PR adds semester joined and college to the user profile. Semester joined is used to keep track of who is a newbie, for coffee chat requirement purposes. College is used on the new website on the pie chart. Semester joined is visible but not editable on the user side -- it should be configured by admin. College is editable on the user side.

Notion/Figma Link

Notion

Test Plan

Editing the fields, verifying values persisted in database.

https://github.com/user-attachments/assets/70723fef-cb8e-48c4-9ba8-e6373739b185

Notes

dti-github-bot commented 1 week ago

[diff-counting] Significant lines: 95.

andrew032011 commented 1 week ago

Semester joined and college have not been populated in the database.

@cchrischen The order here (to prevent downtime on the edit profile page) is to populate semesterJoined field before you merge this in. Either through a script or it might actually be easier to use the "Upload CSV" tool (I can help with this since the tool is a little buggy right now and not super easy to work with: https://www.notion.so/Edit-User-Improvements-1180ad723ce180a88ed2e562ca0ee4ad?pvs=4).