Skip to content

feat: [WIP] Automate GDrive Access for approved volunteers#114

Draft
Mr-Sunglasses wants to merge 2 commits intopyladies:mainfrom
Mr-Sunglasses:feat/109/g-drive
Draft

feat: [WIP] Automate GDrive Access for approved volunteers#114
Mr-Sunglasses wants to merge 2 commits intopyladies:mainfrom
Mr-Sunglasses:feat/109/g-drive

Conversation

@Mr-Sunglasses
Copy link
Copy Markdown
Contributor

fix: #109

@Mariatta This is the first prototype of gdrive acess feature and it is working for me.

  • Did we have to show in the volunteer profile that the user have the access of gdrive ?

@netlify
Copy link
Copy Markdown

netlify bot commented May 6, 2025

Deploy Preview for pyladiescon-portal-docs ready!

Name Link
🔨 Latest commit dbc311a
🔍 Latest deploy log https://app.netlify.com/projects/pyladiescon-portal-docs/deploys/682b8ce41216e30008a55988
😎 Deploy Preview https://deploy-preview-114--pyladiescon-portal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.


# Google Drive Settings
GOOGLE_SERVICE_ACCOUNT_FILE = os.path.join(
BASE_DIR, "krishnapachauri-208910-574aa1e639b3.json"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

use env var instead of hardcoding

GOOGLE_SERVICE_ACCOUNT_FILE = os.path.join(
BASE_DIR, "krishnapachauri-208910-574aa1e639b3.json"
)
GOOGLE_DRIVE_FOLDER_ID = "18P_t0wpEXuMQyQkZJxccsacOh-eelLDk"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Use env var instead of hardcoding

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I’ve make it a test version to verify the logic is correct or not, will fix it once we agree that this is what we gonna do.

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.

Automate some of the volunteer process

2 participants