-
Notifications
You must be signed in to change notification settings - Fork 12.3k
feat(unified-cal): connection-based unified calendar API with CRUD, freebusy, and list connections #28387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+3,522
−95
Merged
feat(unified-cal): connection-based unified calendar API with CRUD, freebusy, and list connections #28387
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
21ae55b
feat(unified-cal): connection-based unified calendar API with CRUD, f…
sahitya-chandra a288fb2
style: apply biome formatting to unified calendar API files
sahitya-chandra 77f649a
fix: use @IsTimeZone() validator for timeZone field in CreateEventDat…
sahitya-chandra 0898c10
fix: add delegation auth support, extract freebusy service layer
sahitya-chandra 7c4ebae
test: add unit and integration tests for unified calendar API
sahitya-chandra 73ae5b2
fix: address Devin Review feedback - fix JSDoc and validator pattern
sahitya-chandra 0c4c44c
fix: revert IsAfterFrom to throw BadRequestException per team convention
sahitya-chandra 29611d9
fix: add calendarId query param to createConnectionEvent for API cons…
sahitya-chandra e18e462
Update apps/api/v2/src/modules/cal-unified-calendars/controllers/cal-…
sahitya-chandra 7cda999
Revert "Update apps/api/v2/src/modules/cal-unified-calendars/controll…
sahitya-chandra a0c6e42
feat: enhance calendar service with connection-specific methods and i…
sahitya-chandra 4718c7e
test: complete delegation auth tests, document virtual mocks, fix key…
sahitya-chandra 4f39e2f
fix: add defense-in-depth key stripping in listConnections controller
sahitya-chandra 56413ae
feat: add unified calendar API endpoints for connections and events m…
sahitya-chandra 51c5a0b
fix: add try/catch error handling to CRUD helper methods
sahitya-chandra a134782
fix: map Google API errors to correct HTTP status codes
sahitya-chandra 3d36df5
fix: preserve upstream Google API status codes in error mapping
sahitya-chandra 721aa75
fix: distinguish Google quota/rate-limit 403 from permission 403
sahitya-chandra bf1ff03
fix: keep dailyLimitExceeded as 403 (non-retriable quota exhaustion)
sahitya-chandra f371f17
fix: add missing @ApiQuery decorators for calendarId on get/update/de…
sahitya-chandra e7d8889
ci: retry flaky vitest worker test
sahitya-chandra db43cb9
fix: update calendarId query parameter to be optional in OpenAPI spec…
sahitya-chandra db50eda
Merge branch 'main' into devin/1773319698-unified-calendar-api
sahitya-chandra 1ffef99
fix: swap dual decorator order so plural /events/ path appears in Ope…
sahitya-chandra 7cc78ca
fix: split dual decorators into separate methods so both paths appear…
sahitya-chandra a5fa1c6
fix: update openapi.json with split dual-decorator paths for GET/PATC…
sahitya-chandra d445bd0
fix: mapGoogleApiError - coerce string code to number and read errors…
sahitya-chandra 84cffd4
fix: mapGoogleApiError - guard against NaN from non-numeric error codes
sahitya-chandra 122b9bb
fix: use read replica for findCredentialWithDelegationByTypeAndUserId…
sahitya-chandra f2800f1
refactor: address review comments - UnifiedCalendarService, ParseConn…
sahitya-chandra 48d2d7d
chore: regenerate openapi.json after controller refactor to array syn…
sahitya-chandra 9e79dfb
Merge branch 'main' into devin/1773319698-unified-calendar-api
sahitya-chandra File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 changes: 6 additions & 2 deletions
8
apps/api/v2/src/modules/cal-unified-calendars/cal-unified-calendars.module.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.