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

✨ Source Microsoft SharePoint: Add parsing shared files #35830

Merged
merged 10 commits into from
Mar 18, 2024

Conversation

tolik0
Copy link
Contributor

@tolik0 tolik0 commented Mar 5, 2024

What

How

Iterating over shared objects is implemented using the requests library, as office365 doesn't support getting items using drive ID and object ID.
Add a new parameter, search_scope, to select where to search for the files.

Fix file paths by adding the drive URL to the file path to differentiate between files from different drives.

Copy link

vercel bot commented Mar 5, 2024

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Mar 13, 2024 9:49pm

@tolik0 tolik0 self-assigned this Mar 6, 2024
@tolik0 tolik0 marked this pull request as ready for review March 6, 2024 11:32
@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Mar 6, 2024
@octavia-squidington-iv octavia-squidington-iv requested a review from a team March 6, 2024 11:33
@tolik0 tolik0 force-pushed the tolik0/source-microsoft-sharepoint/add-shared-drives branch from e037117 to 243ef28 Compare March 6, 2024 12:55
Copy link
Collaborator

@lazebnyi lazebnyi left a comment

Choose a reason for hiding this comment

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

lgtm

minor comment for error message

item_url = f"{base_url}/items/{object_id}"
item_response = requests.get(item_url, headers=headers)
if item_response.status_code != 200:
raise RuntimeError(f"Error retrieving shared object: {item_response.status_code}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

We need specified more user friendly error message with more details.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@tolik0 tolik0 requested a review from lazebnyi March 13, 2024 22:23
Copy link
Collaborator

@lazebnyi lazebnyi left a comment

Choose a reason for hiding this comment

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

lgtm

@tolik0 tolik0 merged commit 80332d8 into master Mar 18, 2024
30 checks passed
@tolik0 tolik0 deleted the tolik0/source-microsoft-sharepoint/add-shared-drives branch March 18, 2024 16:25
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 connectors/source/microsoft-sharepoint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants