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
This commit is contained in:
committed by
GitHub
parent
71724de7dd
commit
21543b6803
+7
-4
@@ -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;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user