Skip to content

revert: "fix: Add new route to create team routing-form response (#22…#22399

Merged
keithwillcode merged 2 commits intomainfrom
revert/team-routing-form-api-change
Jul 11, 2025
Merged

revert: "fix: Add new route to create team routing-form response (#22…#22399
keithwillcode merged 2 commits intomainfrom
revert/team-routing-form-api-change

Conversation

@keithwillcode
Copy link
Contributor

@keithwillcode keithwillcode commented Jul 10, 2025

This reverts commit 89e7fca.


Summary by cubic

Removes the new API route and related logic for creating team routing-form responses, restoring the previous structure and access controls.

  • Refactors
    • Deletes new guards, services, and tests added for team routing-form response creation.
    • Updates controllers, modules, and OpenAPI docs to remove the new route and related types.

@keithwillcode keithwillcode requested a review from a team July 10, 2025 23:07
@keithwillcode keithwillcode requested a review from a team as a code owner July 10, 2025 23:07
@keithwillcode keithwillcode self-assigned this Jul 10, 2025
@vercel
Copy link

vercel bot commented Jul 10, 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) Visit Preview Jul 11, 2025 0:44am
cal-eu ⬜️ Ignored (Inspect) Visit Preview Jul 11, 2025 0:44am

@keithwillcode keithwillcode added core area: core, team members only foundation labels Jul 10, 2025
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 api area: API, enterprise API, access token, OAuth routing-forms area: routing forms, routing, forms labels Jul 10, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jul 10, 2025

No security or compliance issues detected. Reviewed everything up to 2fdf3ab.

Security Overview
  • 🔎 Scanned files: 15 changed file(s)
Detected Code Changes

The diff is too large to display a summary of code changes.

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.

@keithwillcode keithwillcode enabled auto-merge (squash) July 10, 2025 23:12
@graphite-app
Copy link

graphite-app bot commented Jul 10, 2025

Graphite Automations

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

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 10, 2025

E2E results are ready!

@vercel vercel bot temporarily deployed to Preview – api July 11, 2025 00:44 Inactive
@keithwillcode keithwillcode merged commit e6c36af into main Jul 11, 2025
38 checks passed
@keithwillcode keithwillcode deleted the revert/team-routing-form-api-change branch July 11, 2025 01:02
Arephan pushed a commit to Arephan/cal.com that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 11, 2025
hariombalhara added a commit that referenced this pull request Jul 17, 2025
…evert earlier) (#22407)

* Revert "revert: "fix: Add new route to create team routing-form response (#22347)" (#22399)"

This reverts commit e6c36af.

* Update the documentation

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

Labels

api area: API, enterprise API, access token, OAuth core area: core, team members only foundation ready-for-e2e routing-forms area: routing forms, routing, forms

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants