Skip to content

Merge to Main 4/1/2026#481

Open
issacli-0821 wants to merge 4 commits intomainfrom
preview
Open

Merge to Main 4/1/2026#481
issacli-0821 wants to merge 4 commits intomainfrom
preview

Conversation

@issacli-0821
Copy link
Copy Markdown
Collaborator

References

  • Linear: N/A

Proposed Changes

  • Remove logging on prod
  • Fix deletion of user info in YAML insertion
  • Update insert_evals.py

* Update insert_evals to not delete all existing comments

* Remove comment splitting by comma in place of json encoding
* DEV-685: Remove null names in yamls (#475)

* Make name not-nullable for Requirement

* Add names for FIT

* Add names for GER

* Add names for COS-AB

* Add names for GEO

* Implement validate_yaml function to check for duplicate sibling req names

* Call validate_yamls at the start of the script

* Use update_or_create based on parent req when creating requirements

* Remove orphaned requirements

* Remove the deletion of user info

* Log deleted reqs

* Add check for null names in validate_yaml

* Remove unneeded prints

* Move clear user_req_dict to after reqs have been created

* Implemenet helper function to delete orphaned reqs

* Optimize Yaml load to only do it once

* Optimize course_list query by batching query across all courses

* Move cache update inside push functions

* Compile regex at module load

* Only set course list if there is a change

* Optimize delete_orphaned_requirements

* Optimize cache update for degree/major

* Update logging statements
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
plan Ready Ready Preview, Comment Apr 2, 2026 8:15pm

Request Review

@issacli-0821 issacli-0821 requested a review from emiilyyou April 1, 2026 18:47
* Remove logging in InfoComponent

* Remove duplicate instructor setting
* Implement redirect to dashboard after user logs in

* Redirect users to calendar/dashboard when logged out
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.

4 participants