Skip to content

Conversation

@jloh02
Copy link
Member

@jloh02 jloh02 commented Nov 17, 2025

The current scraper does not allow us to easily see when the last scrape was performed. When the cron fails, we should identify this quickly from the endpoint at https://api.nusmods.com/v2/cpexModules.json. This PR adds a new field "last updated" into the JSON output

@vercel
Copy link

vercel bot commented Nov 17, 2025

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

Project Deployment Preview Comments Updated (UTC)
nusmods-export Ready Ready Preview Comment Nov 18, 2025 3:41am
nusmods-website Ready Ready Preview Comment Nov 18, 2025 3:41am

@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.14%. Comparing base (988c6fd) to head (b798f09).
⚠️ Report is 148 commits behind head on master.

Files with missing lines Patch % Lines
website/src/apis/mpe.ts 0.00% 2 Missing ⚠️
website/src/views/mpe/form/ModuleForm.tsx 0.00% 2 Missing ⚠️
website/src/views/mpe/form/MpeFormContainer.tsx 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4254      +/-   ##
==========================================
+ Coverage   54.52%   55.14%   +0.61%     
==========================================
  Files         274      297      +23     
  Lines        6076     6724     +648     
  Branches     1455     1617     +162     
==========================================
+ Hits         3313     3708     +395     
- Misses       2763     3016     +253     

☔ 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.

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.

2 participants