[PR #2236] [MERGED] fix: optimize webhook routing #2142

New issue

Closed

opened 2026-02-26 20:32:36 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 20:32:36 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2236
Author: @dguyen
Created: 11/24/2025
Status: ✅ Merged
Merged: 11/25/2025
Merged by: @Mythie

Base: main ← Head: fix/optimize-webhook-routing

📝 Commits (2)

7a55aba fix: optimize webhook routing
8083e54 fix: remove redundant step

📊 Changes

4 files changed (+55 additions, -65 deletions)

View changed files

📝 apps/remix/app/routes/api+/webhook.trigger.ts (+1 -0)
📝 packages/lib/jobs/definitions/internal/execute-webhook.handler.ts (+40 -42)
📝 packages/lib/server-only/webhooks/trigger/handler.ts (+1 -0)
📝 packages/lib/server-only/webhooks/trigger/trigger-webhook.ts (+13 -23)

📄 Description

Description

Directly execute webhooks in the backend instead of routing it through the frontend API router.

Notes

In the future we should probably have a new job which does webhook fetch, then parallel sends the requests to the client. This would let us also store a "webhook event ID" which would allow us to group WebhookRuns into a single event

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/2236 **Author:** [@dguyen](https://github.com/dguyen) **Created:** 11/24/2025 **Status:** ✅ Merged **Merged:** 11/25/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/optimize-webhook-routing` --- ### 📝 Commits (2) - [`7a55aba`](https://github.com/documenso/documenso/commit/7a55abadaf46c99204d4c8ab7ce2b627f35aa5e9) fix: optimize webhook routing - [`8083e54`](https://github.com/documenso/documenso/commit/8083e54625c9ac7d0c1fde4c14cb31e7b18719a6) fix: remove redundant step ### 📊 Changes **4 files changed** (+55 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/routes/api+/webhook.trigger.ts` (+1 -0) 📝 `packages/lib/jobs/definitions/internal/execute-webhook.handler.ts` (+40 -42) 📝 `packages/lib/server-only/webhooks/trigger/handler.ts` (+1 -0) 📝 `packages/lib/server-only/webhooks/trigger/trigger-webhook.ts` (+13 -23) </details> ### 📄 Description ## Description Directly execute webhooks in the backend instead of routing it through the frontend API router. ## Notes In the future we should probably have a new job which does webhook fetch, then parallel sends the requests to the client. This would let us also store a "webhook event ID" which would allow us to group `WebhookRuns` into a single event --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 20:32:36 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-02-26 20:33:19 +03:00

[PR #2439] fix(i18n): mark page titles for translation #2309

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/documenso#2142

No description provided.

Rows
Columns