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: Fix issue with API key being required for the Qdrant Node #8237

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

Anush008
Copy link
Contributor

@Anush008 Anush008 commented Jan 5, 2024

Summary

This PR makes the API key value in the Qdrant credentials optional, since local Qdrant deployments may not have it configured.

@n8n-assistant n8n-assistant bot added the community Authored by a community member label Jan 5, 2024
@Joffcom Joffcom changed the title chore: make API key optional QdrantApi.credentials.ts fix(Qdrant Node): Fix issue with API key being required Jan 5, 2024
@Joffcom Joffcom changed the title fix(Qdrant Node): Fix issue with API key being required fix: Fix issue with API key being required in the Qdrant Node Jan 9, 2024
@Joffcom Joffcom changed the title fix: Fix issue with API key being required in the Qdrant Node fix: Fix issue with API key being required for the Qdrant Node Jan 9, 2024
@Joffcom Joffcom merged commit 4401db3 into n8n-io:master Jan 9, 2024
9 of 11 checks passed
MiloradFilipovic added a commit that referenced this pull request Jan 10, 2024
* master:
  ci: Validate docs urls for langchain nodes as well (no-changelog) (#8271)
  fix: Small ui fixes to workflow cred setup modal (no-changelog) (#8280)
  fix: Fix issue with API key being required for the Qdrant Node (#8237)
  feat(Coda Node): Add User-Agent for requests to Coda (no-changelog) (#7771)
  fix(FTP Node): FTP connection failed due to missing password credential in node (#8131)
  ci: Fix lint setup in `chat` package (no-changelog) (#8275)
  ci: Fix codecov reporting, and include all packages (no-changelog) (#8276)
  fix(core): Fix test webhook deregistration (#8247)
  docs: Update links in deprecated langchain nodes to avoid linting errors (no-changelog) (#8273)
  fix: Fix user reinvites on FE and BE (#8261)
  docs: Update primaryDocumentation urls for nodes updated in #7651 (no-changelog) (#8270)
  feat: Add Chat Trigger node (#7409)
  fix(Github Trigger Node): Enforce SSL validation by default (#8265)
  fix(editor): Tweaking button sizes in execution preview (#8206)
  fix(core): Avoid wrapping `ExecutionBaseError` to prevent misreporting to Sentry (no-changelog) (#8262)
@github-actions github-actions bot mentioned this pull request Jan 10, 2024
@Anush008 Anush008 deleted the patch-1 branch January 25, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants