n8n-io / n8n

Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
https://n8n.io
Other
49.52k stars 7.98k forks source link

fix(editor): Don't re-render input panel after node finishes executing #11813

Closed RicardoE105 closed 5 days ago

RicardoE105 commented 5 days ago

Summary

Before

https://share.cleanshot.com/VXhQ7rb9

Now

CleanShot 2024-11-20 at 10 41 23

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-2607/bug-previous-node-re-execution-when-it-should-not

Review / Merge checklist

cypress[bot] commented 5 days ago

n8n    Run #8021

Run Properties:  status check passed Passed #8021  •  git commit 9e6c280103: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 RicardoE105 🗃️ e2e/*
Project n8n
Branch Review ado-2607-bug-previous-node-re-execution-when-it-should-not
Run status status check passed Passed #8021
Run duration 04m 33s
Commit git commit 9e6c280103: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 RicardoE105 🗃️ e2e/*
Committer Ricardo Espinoza
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 475
View all changes introduced in this branch ↗︎
github-actions[bot] commented 5 days ago

:white_check_mark: All Cypress E2E specs passed

codecov[bot] commented 5 days ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review. Files with missing lines Patch % Lines
packages/editor-ui/src/components/InputPanel.vue 0.00% 1 Missing :warning:

:loudspeaker: Thoughts on this report? Let us know!