Skip to content

fix: api v2 calendars/busy-times docs#27792

Closed
supalarry wants to merge 4 commits intomainfrom
lauris/cal-7189-fix-api-v2-calendarsget-busy-times-docs
Closed

fix: api v2 calendars/busy-times docs#27792
supalarry wants to merge 4 commits intomainfrom
lauris/cal-7189-fix-api-v2-calendarsget-busy-times-docs

Conversation

@supalarry
Copy link
Contributor

@supalarry supalarry commented Feb 9, 2026

Fixes #27791


Summary by cubic

Updated API v2 /calendars/busy-times docs to require dateFrom/dateTo and calendarsToLoad (bracket-notation array), and clarified that loggedInUsersTz is deprecated in favor of timeZone. This aligns Swagger/OpenAPI and examples with the actual request per CAL-7189.

  • Bug Fixes
    • Added explicit ApiQuery for calendarsToLoad with schema and URL example.
    • Hid nested Calendar fields and calendarsToLoad in Swagger so credentialId/externalId don’t appear as top-level query params.
    • Synced OpenAPI and oasdiff ignore with these changes (removed credentialId/externalId, marked calendarsToLoad required, made dateFrom/dateTo required).

Written for commit f8bb959. Summary will update on new commits.

@supalarry supalarry requested review from a team as code owners February 9, 2026 12:39
@linear
Copy link

linear bot commented Feb 9, 2026

@paragon-review
Copy link

paragon-review bot commented Feb 9, 2026

Paragon Review Unavailable

Hi @supalarry! To enable Paragon reviews on this repository, please register at https://home.polarity.cc

Once registered, connect your GitHub account and Paragon will automatically review your pull requests.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@supalarry supalarry enabled auto-merge (squash) February 10, 2026 11:26
Ryukemeister
Ryukemeister previously approved these changes Feb 11, 2026
@github-actions
Copy link
Contributor

Devin AI is resolving merge conflicts

This PR has merge conflicts with the main branch. A Devin session has been created to automatically resolve them.

View Devin Session

Devin will:

  1. Merge the latest main into this branch
  2. Resolve any conflicts intelligently
  3. Run lint/type checks to ensure validity
  4. Push the resolved changes

If you prefer to resolve conflicts manually, you can close the Devin session and handle it yourself.

@github-actions
Copy link
Contributor

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Feb 20, 2026
@supalarry supalarry closed this Mar 2, 2026
auto-merge was automatically disabled March 2, 2026 13:47

Pull request was closed

@supalarry supalarry deleted the lauris/cal-7189-fix-api-v2-calendarsget-busy-times-docs branch March 2, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: api v2 calendars/get-busy times docs

2 participants