Skip to content

chore(cpex): Update AY25/26 Sem 2 CPEx (for AY26/27 Sem 1)#4331

Merged
jloh02 merged 7 commits intomasterfrom
cpex-staging
Mar 8, 2026
Merged

chore(cpex): Update AY25/26 Sem 2 CPEx (for AY26/27 Sem 1)#4331
jloh02 merged 7 commits intomasterfrom
cpex-staging

Conversation

@jloh02
Copy link
Member

@jloh02 jloh02 commented Mar 5, 2026

Routine updates

@vercel
Copy link

vercel bot commented Mar 5, 2026

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

Project Deployment Actions Updated (UTC)
nusmods-website Ready Ready Preview, Comment Mar 8, 2026 2:03pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
nusmods-export Ignored Ignored Preview Mar 8, 2026 2:03pm

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 5, 2026

Confidence Score: 5/5

  • This PR is safe to merge — all three changes are consistent, mechanical, and follow the same pattern as the previous CPEx update.
  • All changes are narrow, well-scoped, and internally consistent. The schedule dates are correctly formatted with SGT offsets, the academic year string format matches the existing convention, and the feature flags align with the upcoming CPEx start date. No logic changes or new code paths are introduced.
  • No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["CPEx Scraper\n(ACADEMIC_YEAR = '2026/27')"] -->|"Fetches modules for AY26/27"| B["NUS Course API"]
    B -->|"Module list with MPE attributes"| C["cpexModules.json"]
    C -->|"Loaded by website"| D["CPEx Feature\nenableCPExforProd = true\nshowCPExTab = true"]
    D --> E["CPEx Tab visible to users"]
    F["modreg-schedule.json\nCPEx: Mar 9–13, 2026"] -->|"Schedule window"| E
    E -->|"Mar 9, 2026 00:00 SGT"| G["CPEx Opens"]
    G -->|"Mar 13, 2026 23:59 SGT"| H["CPEx Closes"]
    H -->|"Post-CPEx cleanup\n(future PR)"| I["Feature flags reset to false"]
Loading

Last reviewed commit: 2760cea

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.52%. Comparing base (988c6fd) to head (0ee6f2d).
⚠️ Report is 204 commits behind head on master.

Files with missing lines Patch % Lines
website/src/views/mpe/constants.ts 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4331      +/-   ##
==========================================
+ Coverage   54.52%   56.52%   +1.99%     
==========================================
  Files         274      308      +34     
  Lines        6076     6962     +886     
  Branches     1455     1682     +227     
==========================================
+ Hits         3313     3935     +622     
- Misses       2763     3027     +264     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@ravern ravern left a comment

Choose a reason for hiding this comment

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

LGTM

We will only merge this once CPEx is supposed to start.

@jloh02 jloh02 merged commit 034dd4e into master Mar 8, 2026
5 of 6 checks passed
@jloh02 jloh02 deleted the cpex-staging branch March 8, 2026 15:49
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.

3 participants