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): Fix parameter input glitch when there was an error loading remote options #10209

Merged

Conversation

burivuhster
Copy link
Contributor

Summary

This PR disables the input field for a parameter, if there was an error loading remote parameter options.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/AI-237/bug-loadoptions-drop-down-in-openai-model-broken

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)

@burivuhster burivuhster changed the title wip: Fix remote parameter loading error fix(core): Fix parameter input glitch when there was an error loading remote options Jul 26, 2024
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Jul 26, 2024
@burivuhster burivuhster changed the title fix(core): Fix parameter input glitch when there was an error loading remote options fix(editor): Fix parameter input glitch when there was an error loading remote options Jul 26, 2024
@burivuhster burivuhster marked this pull request as ready for review July 26, 2024 12:54
Copy link

cypress bot commented Jul 29, 2024

Passing run #6171 ↗︎

0 389 0 0 Flakiness 0

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 burivuhster 🗃️ e2e/*
Project: n8n Commit: 66032cc5f6
Status: Passed Duration: 04:06 💡
Started: Jul 29, 2024 8:28 AM Ended: Jul 29, 2024 8:33 AM

Review all test suite changes for PR #10209 ↗︎

Copy link
Contributor

✅ All Cypress E2E specs passed

@burivuhster burivuhster merged commit c0e3743 into master Jul 29, 2024
28 checks passed
@burivuhster burivuhster deleted the ai-237-bug-loadoptions-drop-down-in-openai-model-broken branch July 29, 2024 08:50
MiloradFilipovic added a commit that referenced this pull request Jul 30, 2024
* master:
  docs: Update login url for OpenAI node (no-changelog) (#10222)
  fix(LinkedIn Node): Fix issue with some characters cutting off posts early (#10185)
  fix(Google BigQuery Node): Send timeoutMs in query, pagination support (#10205)
  feat(core): Show Public API key value only once (no-changelog) (#10126)
  refactor(core): Display stack trace in error reporter (no-changelog) (#10225)
  fix(core): Fix missing successful items on continueErrorOutput with multiple outputs (#10218)
  fix(n8n Form Trigger Node): Remove custom attribution option (no-changelog) (#10229)
  feat(n8n Form Trigger Node): Improvements (#10092)
  refactor(core): Port path, host, port, listen_address and protocol config (no-changelog) (#10223)
  docs: Update add options text (no-changelog) (#10049)
  fix(Postgres Node): Option to treat query parameters enclosed in single quotas as text (#10214)
  refactor(core): Decouple server started event from internal hooks (no-changelog) (#10221)
  feat(Shopify Node): Update Shopify API version (#10155)
  fix(editor): Defer `User saved credentials` telemetry event for OAuth credentials (#10215)
  fix(editor): Fix parameter input glitch when there was an error loading remote options (#10209)
cstuncsik pushed a commit that referenced this pull request Jul 31, 2024
@github-actions github-actions bot mentioned this pull request Jul 31, 2024
@janober
Copy link
Member

janober commented Jul 31, 2024

Got released with n8n@1.53.0

cstuncsik pushed a commit that referenced this pull request Aug 1, 2024
# for free to join this conversation on GitHub. Already have an account? # 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