Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
This PR adds a start date feature to the schedule generation system, allowing users to specify when their schedule should begin instead of defaulting to the current date. The implementation includes UI components for date selection, API changes to make start date required, and updates to the calendar generation logic.
- Made
startDatea required parameter in both thegenerateIcalandsyncToGoogleCalendarAPI endpoints - Updated the iCal generation logic to calculate event occurrences from the user-specified start date rather than the current date
- Added a date picker UI component with session storage persistence for OAuth redirects
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/server/api/routers/schedule.ts | Changed startDate from optional to required in both API endpoints |
| src/lib/ical.ts | Updated getNextOccurrence to accept a fromDate parameter and renamed local variables to avoid shadowing; fixed same-day event scheduling logic |
| src/components/upload-zone.tsx | Added start date picker UI, state management, session storage persistence, and validation to disable export buttons when date is missing |
| src/components/ui/popover.tsx | New popover component for date picker dropdown |
| src/components/ui/calendar.tsx | New calendar component for date selection |
| src/components/ui/button.tsx | Added default values for variant/size props and data attributes for styling |
| pnpm-lock.yaml | Added dependencies for @radix-ui/react-popover and react-day-picker |
| package.json | Added @radix-ui/react-popover and react-day-picker as dependencies |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.