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): Error dropdown in resource locator disappears when search filter is required #9681

Conversation

elsmr
Copy link
Member

@elsmr elsmr commented Jun 10, 2024

Summary

Dropdown in resource locator disappears on error when search filter is required

Can be tested with the Github Trigger. Should be tested with:

  • No credentials
  • Invalid credentials
  • Valid credentials
image

Related tickets and issues

https://linear.app/n8n/issue/NODE-1206/github-trigger-bug-when-searching-repository-owner-dropdown

Review / Merge checklist

  • PR title and summary are descriptive. Remember, the title automatically goes into the changelog. Use (no-changelog) otherwise. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.

    A bug is not considered fixed, unless a test is added to prevent it from happening again.
    A feature is not complete without tests.

@elsmr elsmr requested a review from MiloradFilipovic June 10, 2024 10:41
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Jun 10, 2024
Copy link
Contributor

@MiloradFilipovic MiloradFilipovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected, code is also looking good 🚀

Copy link
Contributor

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

@MiloradFilipovic MiloradFilipovic self-requested a review June 11, 2024 10:11
Copy link

cypress bot commented Jun 11, 2024

1 flaky test on run #5551 ↗︎

0 395 0 0 Flakiness 1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 elsmr 🗃️ e2e/*
Project: n8n Commit: 9ed9abc9e5
Status: Passed Duration: 04:17 💡
Started: Jun 18, 2024 8:57 AM Ended: Jun 18, 2024 9:01 AM
Flakiness  e2e/10-undo-redo.cy.ts • 1 flaky test

View Output Video

Test Artifacts
Undo/Redo > should undo/redo renaming node using NDV Test Replay Screenshots Video

Review all test suite changes for PR #9681 ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

Copy link
Contributor

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

Copy link
Contributor

✅ All Cypress E2E specs passed

Copy link
Contributor

✅ No visual regressions found.

Copy link
Contributor

✅ All Cypress E2E specs passed

@elsmr elsmr merged commit 1a3f72b into master Jun 18, 2024
30 checks passed
@elsmr elsmr deleted the node-1206-github-trigger-bug-when-searching-repository-owner-dropdown branch June 18, 2024 09:11
adrian-martinez-onestic pushed a commit to onesdata/n8n-fork that referenced this pull request Jun 20, 2024
adrian-martinez-onestic pushed a commit to onesdata/n8n-fork that referenced this pull request Jun 20, 2024
This was referenced Jun 20, 2024
@janober
Copy link
Member

janober commented Jun 20, 2024

Got released with [email protected]

1 similar comment
@janober
Copy link
Member

janober commented Jun 20, 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 ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants