Skip to content

feat: pregenerate location data at build time #1434

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
ronaldwen07 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: pregenerate location data at build time #1434

ronaldwen07 wants to merge 2 commits into from

Conversation

@ronaldwen07
Copy link

@ronaldwen07 ronaldwen07 commented Jan 18, 2026

Summary

  • Move catalogueBuilder to run at preStart/prebuild instead of being static checked-in files
  • This follows the same pattern as PR feat: pregenerate terms #1221 for term data pregeneration
  • Building location data is now fetched from the UCI Map API at build time and written to src/generated/

Test plan

  • Run pnpm run fetch-location-data to verify the script works
  • Run pnpm run build to verify the build completes successfully
  • Verify map functionality still works with the generated location data

Closes #1140

ronald wen added 2 commits January 18, 2026 01:16
fix: show missing section warnings on schedule load (icssc#1399)
66ce27c 
When courses are restored from saved state on page load, missing section
warnings now appear for courses that require additional sections (e.g., lab,
discussion) that weren't saved.

Changes:
- Add checkForMissingSections() method to AppStore that detects and displays
  warnings for courses missing required sections
- Call checkForMissingSections() after successful schedule load
- Extract sectionTypeToName() utility function to eliminate code duplication
- Refactor getMissingSections.ts to use shared utility function

Fixes icssc#1399
feat: pregenerate location data at build time
df44edd 
Move catalogueBuilder to run at preStart/prebuild instead of being
static checked-in files. This follows the same pattern as PR icssc#1221
for term data pregeneration.

Closes icssc#1140
@ronaldwen07 ronaldwen07 mentioned this pull request Jan 18, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

catalogueBuilder should be run preStart

1 participant

@ronaldwen07