Skip to content

DEV-539: Calendar migration#412

Open
issacli-0821 wants to merge 18 commits intopreviewfrom
calendar-migration
Open

DEV-539: Calendar migration#412
issacli-0821 wants to merge 18 commits intopreviewfrom
calendar-migration

Conversation

@issacli-0821
Copy link
Collaborator

@issacli-0821 issacli-0821 commented Feb 7, 2026

References

Proposed Changes

  • Migrate from localStorage to DB for all calendar operations
  • Use localStorage optimistic loading and propagate changes to DB

Testing

  • Check localStorage values prior to migration, and ensure all calendar events are migrated to the DB on loading the calendar page
  • Test with new account without existing calendar data

dl6549 and others added 10 commits January 27, 2026 13:54
* added addCourse implementation

* Changed Default Calendar Name
* removecourse changes

* Remove try-catch and return statements; lint

---------

Co-authored-by: issacli-0821 <122345124+issacli-0821@users.noreply.github.com>
Co-authored-by: Spencer Doyle <124114480+Spencer04Hire@users.noreply.github.com>
Co-authored-by: issacli-0821 <tl1719@princeton.edu>
* activateSection changes

* new changes

* Lint calendarSlice.ts

---------

Co-authored-by: Rishabh Mohapatra <rm5883@princeton.edu>
Co-authored-by: issacli-0821 <tl1719@princeton.edu>
* added react hook to create a new calendar if user does not already have one

* commented out test code

* Update page.tsx

fixed comments

* fixed import call

* pushing for debugging

* bug fixes

* bug fixes

* bug fixes

* automatically add events to calendar upon creation

* merged calendar-migration, optimized async calls

* wrapped calendar iterations in a promise.all

* Modify null checks to fix implementation; import constant for calendar name

* Lint files; delete commented out code

---------

Co-authored-by: issacli-0821 <tl1719@princeton.edu>
@vercel
Copy link

vercel bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
plan Ready Ready Preview, Comment Feb 12, 2026 3:49pm

Request Review

@issacli-0821 issacli-0821 changed the title Calendar migration DEV-539: Calendar migration Feb 7, 2026
@linear
Copy link

linear bot commented Feb 7, 2026

@vercel
Copy link

vercel bot commented Feb 9, 2026

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

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.

6 participants