Skip to content

Conversation

awpala
Copy link
Contributor

@awpala awpala commented Dec 10, 2024

Additional housekeeping tasks:

  • Ran yarn fmt for linting
  • Exported seed to update with new courses in local Firebase Emulators (__seed__/firebase-seed/... files changes)

Copy link

vercel bot commented Dec 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 3:26am

Comment on lines -528 to -533
| 'Quantum Computing'
| 'Introduction to Computing Law'
| 'Digital Health Equity'
| 'Global Entrepreneurship'
| 'GPU Hardware and Software'
| 'Security Incident Response'
Copy link
Contributor Author

@awpala awpala Dec 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These three courses (QC, DHE, SIR) have been given dedicated/non-8803 course sections, corresponding updates have been made elsewhere in this PR (i.e., setting 8803's to isDeprecated: true and "adding" corresponding "new" sections/counterparts with dedicated/permanent course numbers)

@awpala
Copy link
Contributor Author

awpala commented Dec 10, 2024

Note: please do NOT merge immediately on approval; I need to make the corresponding updates (manually) in prod firestore db in tandem with merging into main, so please ping me instead whenever approved and ready to merge

CC: @tran-christian

@awpala awpala merged commit 07d9f3e into main Dec 30, 2024
16 checks passed
@awpala awpala deleted the feature-add-new-courses branch December 30, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants