Skip to content

✨feat(source-microsoft-sharepoint): adjust file record message protocol #57507

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

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

Conversation

aldogonzalez8
Copy link
Contributor

@aldogonzalez8 aldogonzalez8 commented Apr 7, 2025

What

This PR updates the source-microsoft-sharepoint connector to comply with the latest file transfer record protocol.

Resolves: https://github.com/airbytehq/airbyte-internal-issues/issues/12343

How

  • Updated the source-microsoft-sharepoint: connector's implementation to align with the latest protocol specifications.
  • Modified the release information accordingly.

Review guide

1.airbyte-integrations/connectors/source-microsoft-sharepoint/source_microsoft_sharepoint/stream_reader.py: returns file reference and file metadata.

User Impact

  • Internal protocol alignment that ensures consistent file handling across file-based connectors

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@aldogonzalez8 aldogonzalez8 self-assigned this Apr 7, 2025
Copy link

vercel bot commented Apr 7, 2025

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

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2025 5:10pm

@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 14, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 16, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 16, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aaronsteers aaronsteers force-pushed the aldogonzalez8/source-microsoft-sharepoint/adjust-file-record-message-protocol branch from 56eaf6c to 6b1dfc3 Compare April 16, 2025 21:15
@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 16, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 19, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aldogonzalez8
Copy link
Contributor Author

aldogonzalez8 commented Apr 23, 2025

/approve-regression-tests

Check job output.

✅ Approving regression tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants