From 405e26757e2591b42a4bfeedd1fea997fbbb08c9 Mon Sep 17 00:00:00 2001 From: Alex Grozav Date: Thu, 28 Dec 2023 12:59:01 +0200 Subject: [PATCH] fix(editor): Prevent storing pairedItem data inside of pinData (#8173) ## Summary When pinning data after an execution, the pairedItem data gets stored in the pinData field. This creates problems during executions. Before: image After: image ## Related tickets and issues https://linear.app/n8n/issue/PAY-812/wrong-error-shown-when-paireditem-fails-because-of-incorrect-pinned ## Review / Merge checklist - [x] PR title and summary are descriptive. **Remember, the title automatically goes into the changelog. Use `(no-changelog)` otherwise.** ([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md)) - [x] [Docs updated](https://github.com/n8n-io/n8n-docs) or follow-up ticket created. - [x] Tests included. > A bug is not considered fixed, unless a test is added to prevent it from happening again. > A feature is not complete without tests. --- packages/editor-ui/src/stores/workflows.store.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/editor-ui/src/stores/workflows.store.ts b/packages/editor-ui/src/stores/workflows.store.ts index 24c342324c..0817efc952 100644 --- a/packages/editor-ui/src/stores/workflows.store.ts +++ b/packages/editor-ui/src/stores/workflows.store.ts @@ -693,7 +693,7 @@ export const useWorkflowsStore = defineStore(STORES.WORKFLOWS, { } const storedPinData = payload.data.map((item) => - isJsonKeyObject(item) ? item : { json: item }, + isJsonKeyObject(item) ? { json: item.json } : { json: item }, ); this.workflow = {