Skip to content

cleanup(webhooks): remove routing form webhook support#29025

Draft
romitg2 wants to merge 2 commits into
calcom:mainfrom
romitg2:cleanup/remove-routing-form-webhook-support
Draft

cleanup(webhooks): remove routing form webhook support#29025
romitg2 wants to merge 2 commits into
calcom:mainfrom
romitg2:cleanup/remove-routing-form-webhook-support

Conversation

@romitg2
Copy link
Copy Markdown
Member

@romitg2 romitg2 commented Apr 26, 2026

What this does

  • removes the remaining internal webhook plumbing for unsupported routing-form events
  • drops form webhook DTOs, producer queue methods, queued task payloads, and related test scaffolding
  • keeps enum fallback mapping in the payload builder factory until enum/schema cleanup lands

Validation

yarn vitest run packages/features/webhooks/lib/__tests__/consumer/WebhookTaskConsumer.test.ts packages/features/webhooks/lib/service/__tests__/WebhookTaskerProducerService.test.ts packages/features/bookings/lib/handleNewBooking/test/webhook-producer-booking-requested.test.ts

Result: passed (34 tests)

yarn type-check:ci --force

Result: passed

@romitg2 romitg2 added ready-for-e2e run-ci Approve CI to run for external contributors labels Apr 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 26, 2026

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:

Unknown release type "cleanup" found in pull request title "cleanup(webhooks): remove routing form webhook support". 

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

@github-actions github-actions Bot added the ❗️ migrations contains migration files label Apr 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

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 May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

❗️ migrations contains migration files ready-for-e2e run-ci Approve CI to run for external contributors size/L Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant