Skip to content

feat: add organization routing forms endpoints [v2] #20580

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Apr 11, 2025

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Link to Devin run: https://app.devin.ai/sessions/04f0f7f8a7564b32b57725adfcd3167a

This PR implements the following API endpoints for organization routing forms:

  1. GET /v2/organizations/:orgId/routing-forms - Get organization routing forms
  2. GET /v2/organizations/:orgId/teams/:teamId/routing-forms - Get team routing forms
  3. GET /v2/organizations/:orgId/routing-forms/:routingFormId/responses - Get routing form responses
  4. PATCH /v2/organizations/:orgId/routing-forms/:routingFormId/responses/:responseId - Update routing form response

Each endpoint includes:

  • Authentication and authorization guards
  • Input validation
  • Pagination and filtering
  • E2E tests
  • Proper error handling

Requested by: [email protected]

@devin-ai-integration devin-ai-integration bot requested review from a team as code owners April 7, 2025 08:49
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Apr 7, 2025
@graphite-app graphite-app bot requested a review from a team April 7, 2025 08:49
Copy link

vercel bot commented Apr 7, 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 Apr 11, 2025 7:06am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Apr 11, 2025 7:06am

@dosubot dosubot bot added routing-forms area: routing forms, routing, forms ✨ feature New feature or request labels Apr 7, 2025
Copy link

graphite-app bot commented Apr 7, 2025

Graphite Automations

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

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

"Add community label" took an action on this PR • (04/07/25)

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

@ThyMinimalDev ThyMinimalDev marked this pull request as draft April 7, 2025 08:52
@ThyMinimalDev
Copy link
Contributor

Testing Devin, wrote an API spec for endpoints and seeing what's its producing from it

beforeUpdatedAt?: Date;
}
) {
return this.organizationsRoutingFormsRepository.getOrganizationRoutingForms(orgId, skip, take, options);
Copy link
Contributor

Choose a reason for hiding this comment

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

missing plainToClass

ThyMinimalDev
ThyMinimalDev previously approved these changes Apr 10, 2025
ThyMinimalDev
ThyMinimalDev previously approved these changes Apr 11, 2025
@keithwillcode keithwillcode disabled auto-merge April 11, 2025 09:57
@keithwillcode keithwillcode merged commit 3ee8032 into main Apr 11, 2025
39 of 40 checks passed
@keithwillcode keithwillcode deleted the devin/1744015226-routing-forms-endpoints branch April 11, 2025 09:58
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 community Created by Linear-GitHub Sync ✨ feature New feature or request ❗️ migrations contains migration files organizations area: organizations, orgs ready-for-e2e routing-forms area: routing forms, routing, forms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants