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

🎉 New Destination deepset #48875

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

Conversation

abrahamy
Copy link

@abrahamy abrahamy commented Dec 10, 2024

What

This PR adds a new Airbyte destination for deepset Cloud workspaces.

How

The PR uses deepset Cloud's file upload API endpoint to upload unstructured documents as Markdown files to the configured workspace.

The check method uses deepset Cloud's user introspection endpoint (/me) to check if the configured api_key property has access to the target workspace.

The destination is compatible with any sources that produce unstructured document streams. That is, the stream contains content and document_key fields.

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Dec 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Dec 12, 2024 8:36pm

Copy link

vercel bot commented Dec 10, 2024

@abrahamy is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels Dec 10, 2024
@abrahamy abrahamy temporarily deployed to community-ci-auto December 10, 2024 13:44 — with GitHub Actions Inactive
@CLAassistant
Copy link

CLAassistant commented Dec 10, 2024

CLA assistant check
All committers have signed the CLA.

@FHardow
Copy link

FHardow commented Jan 8, 2025

Hey @natikgadzhi - happy new years :)

Could we get a review from your side or could you please tell us how we can continue to get this PR merged?

docs/integrations/destinations/deepset.md Outdated Show resolved Hide resolved
docs/integrations/destinations/deepset.md Outdated Show resolved Hide resolved
docs/integrations/destinations/deepset.md Outdated Show resolved Hide resolved
docs/integrations/destinations/deepset.md Show resolved Hide resolved
@abrahamy
Copy link
Author

Hi @natikgadzhi, please could you or someone on your team take another look at this pr? 😺

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation community connectors/destination/deepset
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants