Skip to content

Upload from Google Drive [Feature]#467

Merged
birm merged 3 commits intocamicroscope:developfrom
akhil-rana:gDrive-upload
Dec 23, 2020
Merged

Upload from Google Drive [Feature]#467
birm merged 3 commits intocamicroscope:developfrom
akhil-rana:gDrive-upload

Conversation

@akhil-rana
Copy link
Copy Markdown
Contributor

@akhil-rana akhil-rana commented Dec 20, 2020

This works with SlideLoader #40 and Distro #154

Description

  • An "upload from google drive" icon is provided during the upload process.
  • Google Picker API is used to let user choose a file from their google drive storage.
  • The picker api will extract the fileID from the chosen file and it'll be sent to Slideloader where Google Drive API will download the corresponding file directly on server-side saving user's time and data.
  • The instructions for the API usage is described in docs/cloudUploadAPIs.md

How Has This Been Tested?

  • Browser: Firefox (v83), Chromium (v87)
  • OS: Windows 10, Arch KDE (Linux-LTS)

Screen-recording Demo :

NOTE:

  • The authentication has to be done twice, once for picker api and another time for drive api. Once they are both done. User will not be prompted again for the same.
  • The app is shown as unverified because it's in testing phase. User can submit their app to google for verification and that message won't be shown anymore.
gDrive.Uplaod.Demo.mp4

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@birm birm merged commit e88e255 into camicroscope:develop Dec 23, 2020
@akhil-rana
Copy link
Copy Markdown
Contributor Author

Thanks a lot for the review. @birm
I hope this feature might come handy for users.

@birm
Copy link
Copy Markdown
Member

birm commented Dec 23, 2020

Thanks doubly for the contribution! I think it will! 🚀

@birm birm mentioned this pull request Jan 25, 2021
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