nusmods icon indicating copy to clipboard operation
nusmods copied to clipboard

Fix for Mobile Google Calendar Odd/Even Week Imports

Open onx001 opened this issue 1 year ago • 4 comments

Added interval checking for numeric week calculations as Google Calendar imports on mobile do not handle exclusions well.

Also fixed additional tutorials for odd/even weeks being added after Week 11/13.

Context

#3679 .ics iCal exports have weird handling issues with even/odd weeks, namely extra lessons after the intended one and a lack of proper exclusion handling for mobile Google Calendar users.

Implementation

A simple interval calculator is implemented for the calculateNumericWeek function, and the count and interval fields are passed to the ics generator where previously they were missing and filled through exclusions.

Other Information

Current handling of edge case intervals that do not follow the regular pattern or that skip intervals might still have the bug for extra lessons. Needs further investigation.

onx001 avatar Mar 24 '24 15:03 onx001

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

Name Status Preview Comments Updated (UTC)
nusmods-export ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2024 0:09am
nusmods-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2024 0:09am

vercel[bot] avatar Mar 24 '24 15:03 vercel[bot]

@onx001 is attempting to deploy a commit to a Personal Account owned by @nusmodifications on Vercel.

@nusmodifications first needs to authorize it.

vercel[bot] avatar Mar 24 '24 15:03 vercel[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.71%. Comparing base (7334b07) to head (1576dd8).

:exclamation: Current head 1576dd8 differs from pull request most recent head ce67073. Consider uploading reports for the commit ce67073 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3684      +/-   ##
==========================================
+ Coverage   53.62%   53.71%   +0.09%     
==========================================
  Files         272      272              
  Lines        5979     5991      +12     
  Branches     1428     1433       +5     
==========================================
+ Hits         3206     3218      +12     
  Misses       2773     2773              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 24 '24 15:03 codecov[bot]

Hi, just tested out this PR as well, and there seems to be another bug: events on odd weeks only show up before recess week, while events on event weeks only show up after recess week.

Here's the timetable I used and I've also attached the exact .ics file I used.

https://nusmods-website-zrekiv0gl-mods-bot.vercel.app/timetable/sem-2/share?ME2115=LAB:2G1,TUT:2G,LEC:1&ME2121=LAB:2G3,TUT:2N,LEC:1

nusmods_calendar.ics.zip

ravern avatar Apr 17 '24 07:04 ravern