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.

Sign up for GitHub

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

Jump to bottom

Load specific file types from Google Drive (issue #4878) #4926

Merged
merged 7 commits into from
May 18, 2023
Merged

Load specific file types from Google Drive (issue #4878) #4926

merged 7 commits into from
May 18, 2023

Conversation

eyurtsev
Copy link
Collaborator

Load specific file types from Google Drive (issue #4878)

Add the possibility to define what file types you want to load from Google Drive.

 loader = GoogleDriveLoader(
    folder_id="1yucgL9WGgWZdM1TOuKkeghlPizuzMYb5",
    file_types=["document", "pdf"]
    recursive=False
)

Fixes ##4878

Who can review?

Community members can review the PR once tests pass. Tag maintainers/contributors who might be interested:
DataLoaders

Twitter: @UmerHAdil | Discord: RicChilligerDude#7589

@eyurtsev
Copy link
Collaborator Author

cc @UmerHA merging in -- made a few minor changes to function signature. looks good!

@eyurtsev eyurtsev mentioned this pull request May 18, 2023
Closed
@eyurtsev eyurtsev merged commit c06a47a into master May 18, 2023
@eyurtsev eyurtsev deleted the eugene/gdrive-filter2 branch May 18, 2023 13:27
@danielchalef danielchalef mentioned this pull request Jun 5, 2023
Merged
This was referenced Jun 15, 2023
Closed
Merged
hwchase17 pushed a commit that referenced this pull request Jun 19, 2023
@tedhli
Iterate through filtered file types instead of all listed files (#6258)
112695e 
# Iterate through filtered file types instead of all listed files

Fixes #6257

#4926 originally added the
functionality to filter by file type, storing the filtered files in
`_files`

#5220 removed the
functionality when adding code to filter trashed files by using the
`files` variables instead of the `_files` variable.

This PR simply adds the functionality back by using `_files` again.

#### Who can review?

@hwchase17 - project lead
@eyurtsev
This was referenced Jun 25, 2023
Closed
Merged
@IanRogers-101Ways IanRogers-101Ways mentioned this pull request Aug 8, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eyurtsev @UmerHA