Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Throw error when failing to save workflow history version #7706

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

valya
Copy link
Contributor

@valya valya commented Nov 14, 2023

Github issue / Community forum post (link here to close automatically):

@valya valya requested a review from ivov November 14, 2023 10:58
@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Nov 14, 2023
ivov
ivov previously approved these changes Nov 15, 2023
Copy link

cypress bot commented Nov 15, 2023

4 flaky tests on run #2882 ↗︎

0 277 3 0 Flakiness 4

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 valya 🗃️ e2e/*
Project: n8n Commit: e9b3adf499
Status: Passed Duration: 05:54 💡
Started: Nov 15, 2023 9:41 AM Ended: Nov 15, 2023 9:47 AM
Flakiness  12-canvas.cy.ts • 1 flaky test

View
Output
Video

Test Artifacts
Canvas Node Manipulation and Navigation > should add nodes and check execution success Screenshots Video
Flakiness  6-code-node.cy.ts • 1 flaky test

View
Output
Video

Test Artifacts
... > generate code button should have correct state & tooltips Screenshots Video
Flakiness  28-resource-mapper.cy.ts • 2 flaky tests

View
Output
Video

Test Artifacts
Resource Mapper > should correctly delete single field Screenshots Video
Resource Mapper > should correctly delete all fields Screenshots Video

Review all test suite changes for PR #7706 ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

@krynble
Copy link
Contributor

krynble commented Nov 28, 2023

I've implemented this PR to try and address an identified issue. I still like the changes here so I think it's worth merging both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants