From 21543b68038ee6685637a5d61276d28c7ae76468 Mon Sep 17 00:00:00 2001 From: Baptiste Devessier Date: Tue, 25 Nov 2025 17:37:32 +0100 Subject: [PATCH] Don't recompute output schema when moving trigger (#16061) Position changes don't affect the trigger's output schema, so recomputing it is unnecessary. For webhook triggers specifically, the backend can't reconstruct the schema (it returns `{}`), which overwrites the frontend-built schema from `expectedBody`. Passing `{ computeOutputSchema: false }` prevents this data loss while avoiding wasteful computation. ## Demo https://github.com/user-attachments/assets/116af221-0d09-4dda-ad40-3301fa610bcd Closes https://github.com/twentyhq/twenty/issues/15982 --- .../components/WorkflowDiagramCanvasEditable.tsx | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/packages/twenty-front/src/modules/workflow/workflow-diagram/components/WorkflowDiagramCanvasEditable.tsx b/packages/twenty-front/src/modules/workflow/workflow-diagram/components/WorkflowDiagramCanvasEditable.tsx index 297b4a69b5..eada4f0fed 100644 --- a/packages/twenty-front/src/modules/workflow/workflow-diagram/components/WorkflowDiagramCanvasEditable.tsx +++ b/packages/twenty-front/src/modules/workflow/workflow-diagram/components/WorkflowDiagramCanvasEditable.tsx @@ -132,10 +132,13 @@ export const WorkflowDiagramCanvasEditable = () => { const triggerToUpdate = workflowWithCurrentVersion?.currentVersion?.trigger; if (isDefined(triggerToUpdate)) { - await updateTrigger({ - ...triggerToUpdate, - position: node.position, - }); + await updateTrigger( + { + ...triggerToUpdate, + position: node.position, + }, + { computeOutputSchema: false }, + ); return; }