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(editor): Fix workflows filter resetting #8411

Conversation

cstuncsik
Copy link
Contributor

Description

On /workflows, tags are no longer clickable (i.e. to apply a filter) after user has reset filters 1x time.

Expected

User can continue to click on a tag to add it to filter criteria, even after they've cleared filters during their session.

Actual

filter-by-tag-bug.mov

Steps or workflow to reproduce (with screenshots/recordings)

n8n version: [cloud prod] [1.9.0]

Open /workflows

Click on a tag

Click "remove filters"

Try clicking on a tag again

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Jan 22, 2024
Copy link
Contributor

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

Copy link

cypress bot commented Jan 25, 2024

13 failed and 1 flaky tests on run #3929 ↗︎

13 170 5 0 Flakiness 1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 MiloradFilipovic 🗃️ e2e/*
Project: n8n Commit: ff7069c8ff
Status: Failed Duration: 04:53 💡
Started: Jan 26, 2024 11:25 AM Ended: Jan 26, 2024 11:30 AM
Failed  5-ndv.cy.ts • 2 failed tests

View Output Video

Test Artifacts
NDV > should allow editing code in fullscreen in the Code node Screenshots Video
NDV > Should keep RLC values after operation change Screenshots Video
Failed  16-webhook-node.cy.ts • 5 failed tests

View Output Video

Test Artifacts
Webhook Trigger node > should listen for a GET request and respond with Respond to Webhook node Test Replay Screenshots Video
Webhook Trigger node > should listen for a GET request and respond with last node Test Replay Screenshots Video
Webhook Trigger node > should listen for a GET request and respond with last node binary data Test Replay Screenshots Video
Webhook Trigger node > should listen for a GET request with Basic Authentication Test Replay Screenshots Video
Webhook Trigger node > should listen for a GET request with Header Authentication Test Replay Screenshots Video
Failed  2-credentials.cy.ts • 2 failed tests

View Output Video

Test Artifacts
Credentials > should show multiple credential types in the same dropdown Test Replay Screenshots Video
Credentials > should rename credentials from NDV Test Replay Screenshots Video
Failed  28-debug.cy.ts • 1 failed test

View Output Video

Test Artifacts
Debug > should be able to debug executions Test Replay Screenshots Video
Failed  16-form-trigger-node.cy.ts • 1 failed test

View Output Video

Test Artifacts
n8n Form Trigger > should fill up form fields Test Replay Screenshots Video

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

Flakiness  cypress/e2e/5-ndv.cy.ts • 1 flaky test

View Output Video

Test Artifacts
NDV > should not retrieve remote options when required params throw errors Screenshots Video

Review all test suite changes for PR #8411 ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

@cstuncsik cstuncsik merged commit ad4b298 into master Jan 26, 2024
28 checks passed
@cstuncsik cstuncsik deleted the pay-876-filter-by-clicking-on-wf-tag-stops-working-after-resetting branch January 26, 2024 05:55
MiloradFilipovic added a commit that referenced this pull request Jan 26, 2024
* master:
  fix(editor): Fix workflows filter resetting (#8411)
  fix(editor): Show pin button on binary output but disable it with tooltip (#8388)
  fix(Merge Node): Passing on no items to "Input 2" results in wrong output items (#8438)
  ci: Enforce `noUnusedLocals` on all backend and nodes packages (no-changelog) (#8428)
  ci: Move docs check notification to #alerts-build (no-changelog) (#8435)
  🚀 Release 1.26.0 (#8427)
@github-actions github-actions bot mentioned this pull request Jan 31, 2024
@janober
Copy link
Member

janober commented Feb 2, 2024

Got released with [email protected]

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 Released ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants