Skip to content

Comments

Revert "feat: Add cancelledBy input to cancellation page"#22479

Merged
anikdhabal merged 1 commit intomainfrom
revert-18770-fix-cal-5045
Jul 14, 2025
Merged

Revert "feat: Add cancelledBy input to cancellation page"#22479
anikdhabal merged 1 commit intomainfrom
revert-18770-fix-cal-5045

Conversation

@anikdhabal
Copy link
Contributor

@anikdhabal anikdhabal commented Jul 14, 2025

Reverts #18770


Summary by cubic

Removed the email verification step and cancelledBy input from the cancellation page, reverting the booking cancellation flow to its previous state. This simplifies the user experience when cancelling a booking.

@linear
Copy link

linear bot commented Jul 14, 2025

@graphite-app graphite-app bot requested a review from a team July 14, 2025 12:44
@github-actions
Copy link
Contributor

github-actions bot commented Jul 14, 2025

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "Revert "feat: Add cancelledBy input to cancellation page"". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@keithwillcode keithwillcode added the core area: core, team members only label Jul 14, 2025
@vercel
Copy link

vercel bot commented Jul 14, 2025

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Jul 14, 2025 0:44am
cal-eu ⬜️ Ignored (Inspect) Jul 14, 2025 0:44am

@graphite-app
Copy link

graphite-app bot commented Jul 14, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (07/14/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (07/14/25)

1 label was added to this PR based on Keith Williams's automation.

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.

cubic reviewed this PR and found no issues. Review PR in cubic.dev.

@dosubot dosubot bot added the bookings area: bookings, availability, timezones, double booking label Jul 14, 2025
@anikdhabal anikdhabal enabled auto-merge (squash) July 14, 2025 12:45
@delve-auditor
Copy link

delve-auditor bot commented Jul 14, 2025

No security or compliance issues detected. Reviewed everything up to 0028b0f.

Security Overview
  • 🔎 Scanned files: 3 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► CancelBooking.tsx
    Remove email verification dialog and simplify cancellation flow
► bookings-single-view.tsx
    Remove attendees and userEmail from booking info
► common.json
    Remove email verification related translations

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@github-actions
Copy link
Contributor

E2E results are ready!

@anikdhabal anikdhabal merged commit 13739da into main Jul 14, 2025
102 of 112 checks passed
@anikdhabal anikdhabal deleted the revert-18770-fix-cal-5045 branch July 14, 2025 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bookings area: bookings, availability, timezones, double booking core area: core, team members only ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants