Skip to content

chore: cron workflows / webhooks API to app router#19875

Merged
hbjORbj merged 3 commits intomainfrom
chore/workflows-api-migration
Mar 10, 2025
Merged

chore: cron workflows / webhooks API to app router#19875
hbjORbj merged 3 commits intomainfrom
chore/workflows-api-migration

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Mar 9, 2025

What does this PR do?

A total of 4 api routes:

  • Migrate schedule APIs in api/cron/workflows
  • Migrate api/cron/webhookTriggers

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Please use the latest Vercel preview and test please 🙏.

@graphite-app graphite-app Bot requested a review from a team March 9, 2025 15:49
@keithwillcode keithwillcode added consumer core area: core, team members only labels Mar 9, 2025
@dosubot dosubot Bot added api area: API, enterprise API, access token, OAuth workflows area: workflows, automations labels Mar 9, 2025
@hbjORbj hbjORbj changed the title chore: workflows api migration chore: cron workflows / webhooks API to app router Mar 9, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Mar 9, 2025

Graphite Automations

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

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

@@ -1,23 +0,0 @@
/* Cron job for scheduled webhook events triggers */
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

entire code is moved to /api/cron/webhookTriggers

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 9, 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 Mar 10, 2025 1:47pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Mar 10, 2025 1:47pm

@hbjORbj hbjORbj enabled auto-merge (squash) March 10, 2025 13:42
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon left a comment

Choose a reason for hiding this comment

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

ALso LGTM - just waiting for all green :)

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 consumer core area: core, team members only ready-for-e2e workflows area: workflows, automations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants