Skip to content

fix: [CAL-1765] opt-in: clicking "confirm" or "reject" broken#9443

Merged
PeerRich merged 4 commits into
mainfrom
hotfix/1765-opt-in-clicking-confirm-or-reject-broken
Jun 10, 2023
Merged

fix: [CAL-1765] opt-in: clicking "confirm" or "reject" broken#9443
PeerRich merged 4 commits into
mainfrom
hotfix/1765-opt-in-clicking-confirm-or-reject-broken

Conversation

@roae
Copy link
Copy Markdown
Contributor

@roae roae commented Jun 9, 2023

What does this PR do?

Add support to a custom sessionGetter in trpc createContext

Fixes #9110

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Create a new booking to a user with an email address you have access to
  • Click on the confirm, or reject button of the email
  • You must get a satisfactory message according to the action (confirm or reject).

Mandatory Tasks

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

@linear
Copy link
Copy Markdown

linear Bot commented Jun 9, 2023

CAL-1765 opt-in: clicking "confirm" or "reject" broken

both buttons just goes to the booking page view

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 9, 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 10, 2023 3:02am
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 3:02am
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 3:02am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
ui ⬜️ Ignored (Inspect) Visit Preview Jun 10, 2023 3:02am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 9, 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 9, 2023

Current Playwright Test Results Summary

✅ 107 Passing - ⚠️ 3 Flaky

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

(Last updated on 06/10/2023 02:52:56am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: f67a5d8

Started: 06/10/2023 02:51:07am UTC

⚠️ Flakes

📄   apps/web/playwright/managed-event-types.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Managed Event Types tests Can create managed event type
Retry 1Initial Attempt
6.92% (20) 20 / 289 runs
failed over last 7 days
21.45% (62) 62 / 289 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
2.05% (6) 6 / 293 runs
failed over last 7 days
23.21% (68) 68 / 293 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
5.18% (10) 10 / 193 runs
failed over last 7 days
94.82% (183) 183 / 193 runs
flaked over last 7 days

View Detailed Build Results


Copy link
Copy Markdown
Contributor

@Udit-takkar Udit-takkar left a comment

Choose a reason for hiding this comment

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

LGTM. Working fine

Tested by

  1. Booked a requires confirmation event type
  2. opened the email confirm link on another browser

@PeerRich PeerRich added this pull request to the merge queue Jun 10, 2023
Merged via the queue into main with commit 0c68e51 Jun 10, 2023
@PeerRich PeerRich deleted the hotfix/1765-opt-in-clicking-confirm-or-reject-broken branch June 10, 2023 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working High priority Created by Linear-GitHub Sync 📉 regressing This used to work. Now it doesn't anymore.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-1765] opt-in: clicking "confirm" or "reject" broken

3 participants