Skip to content

docs: Update API documentation for Bookings endpoint (_post & _patch)#9511

Merged
alishaz-polymath merged 6 commits into
mainfrom
fix/booking-api-swagger
Jun 19, 2023
Merged

docs: Update API documentation for Bookings endpoint (_post & _patch)#9511
alishaz-polymath merged 6 commits into
mainfrom
fix/booking-api-swagger

Conversation

@alishaz-polymath
Copy link
Copy Markdown
Member

What does this PR do?

Updates the API documentation for Bookings endpoint to correctly consider the Booking Questions and replace the deprecated Custom Inputs from it

Type of change

  • Chore (refactoring code, technical debt, workflow improvements)

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 14, 2023

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

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:54pm
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:54pm
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:54pm
web-staging 🛑 Canceled (Inspect) Jun 19, 2023 1:54pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
ui ⬜️ Ignored (Inspect) Visit Preview Jun 19, 2023 1:54pm

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 14, 2023

Current Playwright Test Results Summary

✅ 98 Passing - ⚠️ 2 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 06/19/2023 01:59:15pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 4824a0d

Started: 06/19/2023 01:57:12pm UTC

⚠️ Flakes

📄   apps/web/playwright/reschedule.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Reschedule Tests -- old-booker Attendee should be able to reschedule a booking
Retry 1Initial Attempt
0% (0) 0 / 268 runs
failed over last 7 days
7.84% (21) 21 / 268 runs
flaked over last 7 days

📄   apps/web/playwright/auth/delete-account.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Can delete user account
Retry 1Initial Attempt
2.01% (6) 6 / 298 runs
failed over last 7 days
24.83% (74) 74 / 298 runs
flaked over last 7 days

View Detailed Build Results


@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 14, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@keithwillcode keithwillcode requested a review from a team June 14, 2023 18:28
Comment thread apps/api/pages/api/bookings/_post.ts Outdated
Comment thread apps/api/pages/api/bookings/_post.ts Outdated
Comment thread apps/api/pages/api/bookings/_post.ts Outdated
Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Some nits, otherwise approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only Medium priority Created by Linear-GitHub Sync

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants