Skip to content

Commit bee7267

Browse files
authored
fix(editor): Show NDV errors when opening existing nodes with errors (#12567)
1 parent 88659d8 commit bee7267

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

Diff for: packages/editor-ui/src/composables/useCanvasOperations.test.ts

+12
Original file line numberDiff line numberDiff line change
@@ -995,6 +995,18 @@ describe('useCanvasOperations', () => {
995995

996996
expect(ndvStore.activeNodeName).toBe('Existing Node');
997997
});
998+
999+
it('should set node as dirty when node is set active', () => {
1000+
const workflowsStore = mockedStore(useWorkflowsStore);
1001+
const node = createTestNode();
1002+
1003+
workflowsStore.getNodeById.mockImplementation(() => node);
1004+
1005+
const { setNodeActive } = useCanvasOperations({ router });
1006+
setNodeActive(node.id);
1007+
1008+
expect(workflowsStore.setNodePristine).toHaveBeenCalledWith(node.name, false);
1009+
});
9981010
});
9991011

10001012
describe('setNodeActiveByName', () => {

Diff for: packages/editor-ui/src/composables/useCanvasOperations.ts

+1
Original file line numberDiff line numberDiff line change
@@ -381,6 +381,7 @@ export function useCanvasOperations({ router }: { router: ReturnType<typeof useR
381381
return;
382382
}
383383

384+
workflowsStore.setNodePristine(node.name, false);
384385
setNodeActiveByName(node.name);
385386
}
386387

0 commit comments

Comments
 (0)