Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(editor): Improve configurable nodes design on new canvas #12317

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

Conversation

alexgrozav
Copy link
Member

@alexgrozav alexgrozav commented Dec 19, 2024

Summary

Screenshot 2024-12-19 at 17 35 02

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/CAT-445/configurable-nodes-dont-display-the-same-as-the-old-canvas

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)

@alexgrozav alexgrozav self-assigned this Dec 19, 2024
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Dec 19, 2024
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Copy link
Contributor

⚠️ Some Cypress E2E specs are failing, please fix them before merging

Copy link

cypress bot commented Dec 20, 2024

n8n    Run #8439

Run Properties:  status check failed Failed #8439  •  git commit 22fbd276fc: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 alexgrozav 🗃️ e2e/*
Project n8n
Branch Review cat-445-configurable-nodes-dont-display-the-same-as-the-old-canvas
Run status status check failed Failed #8439
Run duration 04m 38s
Commit git commit 22fbd276fc: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 alexgrozav 🗃️ e2e/*
Committer Alex Grozav
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
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 9
View all changes introduced in this branch ↗︎

Tests for review

Failed  14-mapping.cy.ts • 1 failed test

View Output Video

Test Artifacts
Data mapping > maps expressions from json view Test Replay Screenshots Video
Failed  5-ndv.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  13-pinning.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  12-canvas.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  17-sharing.cy.ts • 0 failed tests

View Output

Test Artifacts

The first 5 failed specs are shown, see all 65 specs in Cypress Cloud.

Copy link
Contributor

⚠️ Some Cypress E2E specs are failing, please fix them before merging

1 similar comment
Copy link
Contributor

⚠️ Some Cypress E2E specs are failing, please fix them before merging

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants