Skip to content

Implemented permanent AWS storage for Genuary#1151

Draft
tmbkoren wants to merge 1 commit into
mainfrom
genuary-aws
Draft

Implemented permanent AWS storage for Genuary#1151
tmbkoren wants to merge 1 commit into
mainfrom
genuary-aws

Conversation

@tmbkoren

Copy link
Copy Markdown
Collaborator
  • Created a module s3handler.js to handle uploading and deleting files from S3 bucket.

  • Created a helper function downloadFile to download pinned items to a temp folder on a runner in order to upload it to S3

  • Updated getMediaFromPins function to download and upload each pin, and append the resulting permanent link to the output.

  • Updated update_genuary workflow to install dependencies and include env variables.

This obviously won't work immediately, because a new AWS S3 bucket with proper permissions needs to be set up first, and credentials fed to the CI/CD pipeline.

Required env variables:

AWS_ACCESS_KEY_ID
AWS_SECRET_ACCESS_KEY
AWS_REGION
AWS_S3_BUCKET_NAME

@netlify

netlify Bot commented Dec 31, 2024

Copy link
Copy Markdown

Deploy Preview for acmcsuf ready!

Name Link
🔨 Latest commit b47db7e
🔍 Latest deploy log https://app.netlify.com/sites/acmcsuf/deploys/67737937b064e50008733fcc
😎 Deploy Preview https://deploy-preview-1151--acmcsuf.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 site configuration.

@EthanThatOneKid EthanThatOneKid left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I haven't tested the code locally myself, however at first glance the code is well-written and looks good to me.

@TheJolman

Copy link
Copy Markdown
Collaborator

if a feature like this ever gets merged we should probably use Cloudflare R2 (10 GB included in free tier)

@tmbkoren

tmbkoren commented Feb 2, 2026

Copy link
Copy Markdown
Collaborator Author

I am not planning to keep working on that since it's been over a year, but anyone is free to take over

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants