Skip to content

Conversation

Tuukkaa
Copy link
Contributor

@Tuukkaa Tuukkaa commented Sep 1, 2025

Summary

Separates node execution errors from validation errors in the canvas UI and removes redundant spinner from waiting state.

Changes:

  • Split issues.items into separate issues.execution and issues.validation arrays
  • Added new node-validation-error icon to design system
  • Updated canvas components to display appropriate icons for each error type
  • Removed extra spinner from node waiting state
  • Enhanced canvas mapping logic to handle separate error states

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/DS-208/separate-state-for-node-execution-and-validation-errors

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Sep 1, 2025
Copy link

bundlemon bot commented Sep 1, 2025

BundleMon

Unchanged files (2)
Status Path Size Limits
WASM Dependencies
tree-sitter-bash.wasm
181.26KB -
WASM Dependencies
tree-sitter.wasm
74.47KB -

No change in files bundle size

Groups updated (2)
Status Path Size Limits
**/*.js
5.73MB (+367.77KB +6.69%) -
**/*.css
195.21KB (+3.96KB +2.07%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

Copy link

codecov bot commented Sep 1, 2025

Codecov Report

❌ Patch coverage is 75.86207% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...nodes/render-types/parts/CanvasNodeStatusIcons.vue 15.38% 11 Missing ⚠️
...tend/editor-ui/src/composables/useCanvasMapping.ts 90.32% 2 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant