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 S3 integration for file uploads with CarrierWave #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Grigore-George-Mihai
Copy link
Owner

Summary

  • This PR integrates CarrierWave and fog-aws to enable file uploads to Amazon S3.
  • It configures the application to use local storage in development and S3 in production.

Related Issue(s)

  • No

Changes

  • Bug fix
  • New feature
  • Code refactor
  • Documentation update
  • Other (please describe):

Description of Changes

  • CarrierWave Configuration: Set up CarrierWave to handle file uploads, using local storage in development and S3 in production.
  • S3 Integration: Configured fog-aws for communication with S3, including handling credentials via environment variables.

How to Test

Steps to test the changes introduced in this PR (e.g., commands to run, environment setup, etc.).

  1. In development, verify files are stored locally by uploading through the UI.
  2. In production, ensure files are uploaded to the specified S3 bucket.
  3. Run and check Rspec

Screenshots (Optional)

If applicable, include screenshots to help explain your changes visually.

Checklist

  • I have run rubocop and fixed any linter warnings.
  • I have run the test suite to ensure everything is passing.
  • I have added tests for my changes.
  • I have updated documentation as needed.

Additional Context (Optional)

Any other relevant information or context.

@Grigore-George-Mihai Grigore-George-Mihai added the enhancement New feature or request label Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant