Skip to content

feat: Enable new booker globally for 3.0#9541

Merged
zomars merged 4 commits into
mainfrom
feat/enable-new-booker-globally
Jun 15, 2023
Merged

feat: Enable new booker globally for 3.0#9541
zomars merged 4 commits into
mainfrom
feat/enable-new-booker-globally

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon commented Jun 15, 2023

  • Enables new-booker with the following env variables. It enables New Booker for non-embed only.
NEW_BOOKER_ENABLED_FOR_NON_EMBED=1 
NEW_BOOKER_ENABLED_FOR_EMBED=0
  • Toggles ON booker-layout feature by default

TODO

  • Test New Booker on Prod with embeds(using the /api/newbooker/enable endpoint)
  • Test some of the known embeds in productions with the new booker and then set NEW_BOOKER_ENABLED_FOR_EMBED=1
  • After this has been in production for some time. Remove old booker code.
CleanShot 2023-06-15 at 14 27 46@2x CleanShot 2023-06-15 at 14 27 30@2x

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 15, 2023

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

Name Status Preview Comments Updated (UTC)
api ❌ Failed (Inspect) Jun 15, 2023 4:40pm
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 15, 2023 4:40pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 15, 2023 4:40pm
web-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 15, 2023 4:40pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 15, 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 15, 2023

Current Playwright Test Results Summary

✅ 114 Passing - ⚠️ 5 Flaky

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

(Last updated on 06/15/2023 04:35:45pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 58891c8

Started: 06/15/2023 04:33:16pm UTC

⚠️ Flakes

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
user can login & logout succesfully login flow user & logout using dashboard
Retry 2Retry 1Initial Attempt
6.18% (16) 16 / 259 runs
failed over last 7 days
27.03% (70) 70 / 259 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Reschedule Tests -- new-booker Paid rescheduling should go to success page
Retry 1Initial Attempt
1.21% (3) 3 / 247 runs
failed over last 7 days
1.21% (3) 3 / 247 runs
flaked over last 7 days

📄   packages/app-store/routing-forms/playwright/tests/basic.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Routing Forms Seeded Routing Form Routing Link - Reporting and CSV Download
Retry 2Retry 1Initial Attempt
0% (0) 0 / 146 runs
failed over last 7 days
10.27% (15) 15 / 146 runs
flaked over last 7 days

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to disable 2FA
Retry 1Initial Attempt
5.12% (13) 13 / 254 runs
failed over last 7 days
20.47% (52) 52 / 254 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
6.45% (10) 10 / 155 runs
failed over last 7 days
93.55% (145) 145 / 155 runs
flaked over last 7 days

View Detailed Build Results


Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

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

Ship it

@zomars zomars enabled auto-merge (squash) June 15, 2023 16:28
@zomars zomars merged commit bf3f8ac into main Jun 15, 2023
@zomars zomars deleted the feat/enable-new-booker-globally branch June 15, 2023 16:36
iamr-kumar pushed a commit to iamr-kumar/cal.com that referenced this pull request Jun 18, 2023
* Update migration to be on by default - remove cookie check

* Use env variable to control enabling NEW_BOOKER for all

* Update packages/prisma/migrations/20230524105015_added_newbooker_feature_flag/migration.sql

* Update turbo.json

---------

Co-authored-by: Hariom Balhara <hariombalhara@gmail.com>
Co-authored-by: Omar López <zomars@me.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

❗️ .env changes contains changes to env variables ❗️ migrations contains migration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants