googlearchive / PyDrive

Google Drive API Python wrapper library
Other
1.31k stars 272 forks source link

Fix 103: simplify fetch all metadata #227

Closed shcheklein closed 3 years ago

shcheklein commented 3 years ago

There is a bug in the current implementation (, is missing between two fields in the _ALL_FIELDS list).

This is an attempt to simplify the logic by using * instead of _ALL_FIELDS + add tests to cover this and check presence some of the fields (e.g. permissions that are not fetched by default if * is not provided).

google-cla[bot] commented 3 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.