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

add more documentation to auth and file management #233

Merged
merged 3 commits into from
Oct 4, 2022
Merged

add more documentation to auth and file management #233

merged 3 commits into from
Oct 4, 2022

Conversation

matiasrebori
Copy link
Contributor

Description

Recently I needed to use Google Drive with Python so this library was very useful but I needed more help for my uses cases that arent the simplest or common, I want to extend the documentation to make more easy its use for future users.

What has been done

  • Oauth: with google service account
  • File management: Get files by complex queries with q parameters
  • File management: Upload data as bytes in memory buffer
  • File management: Upload file to specific folder

Note

See #230 for more insight on the motivation for this.

docs/oauth.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/oauth.rst Outdated Show resolved Hide resolved
docs/oauth.rst Outdated Show resolved Hide resolved
docs/oauth.rst Outdated Show resolved Hide resolved
Copy link
Member

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

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

Thanks a lot @matiasrebori ! We need to iterate on this a bit, but it's a very valuable already

docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/oauth.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
docs/filemanagement.rst Outdated Show resolved Hide resolved
Copy link
Member

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

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

Minor things, good stuff @matiasrebori

@matiasrebori
Copy link
Contributor Author

matiasrebori commented Sep 30, 2022

Minor things, good stuff @matiasrebori

thank you @shcheklein and sorry if i make too much trouble haha. I'll wait for your answers to push a commit

@shcheklein
Copy link
Member

thank you @shcheklein and sorry if i make too much trouble haha. I'll wait for your answers to push a commit

no worries, @matiasrebori !

@matiasrebori matiasrebori temporarily deployed to external October 4, 2022 11:51 Inactive
@shcheklein shcheklein merged commit 368aae0 into iterative:main Oct 4, 2022
@shcheklein
Copy link
Member

Thanks @matiasrebori ! It's merged and should be published now.

@matiasrebori
Copy link
Contributor Author

Thanks to you too @shcheklein, i'm happy to help.

@matiasrebori matiasrebori deleted the documentation branch October 5, 2022 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants