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 fog_provider to carrierwave template #322

Merged

Conversation

andywenk
Copy link
Contributor

when not including the config.fog_provider = 'fog/aws' starting the Rails application will result in the error

.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/activesupport-5.1.4/lib/active_support/dependencies.rb:292:in `require': cannot load such file -- fog (LoadError)

when not including the config.fog_provider = 'fog/aws' starting the Rails application will result in the error

.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/activesupport-5.1.4/lib/active_support/dependencies.rb:292:in `require': cannot load such file -- fog (LoadError)
@andywenk
Copy link
Contributor Author

oh I am not sure why CI fails. This has nothing to do with my change I hope:

Failure:
Spina::PagesControllerTest#test_visit_homepage [/home/ubuntu/Spina/test/functional/spina/pages_controller_test.rb:12]
Minitest::Assertion: Expected response to be a <2XX: success>, but was a <404: Not Found>

@dankmitchell
Copy link
Member

Is this for when you use the fog-aws gem instead of the generic fog gem?

@andywenk
Copy link
Contributor Author

andywenk commented Nov 15, 2017

yes I am using the fog-aws gem. It is not recommended to use the fog gem if you do know the provider. And I found the config here: https://github.com/carrierwaveuploader/carrierwave#using-amazon-s3

@dankmitchell
Copy link
Member

Nice work 👍

@Bramjetten
Copy link
Contributor

Nice find, merging it. Small aside: we will try switching to ActiveStorage soon.

CI is failing because of some weird issue in our tests, rebuilding it fixes it.

@Bramjetten Bramjetten closed this Nov 15, 2017
@Bramjetten Bramjetten reopened this Nov 15, 2017
@Bramjetten Bramjetten merged commit ebb9bc6 into SpinaCMS:master Nov 15, 2017
@andywenk
Copy link
Contributor Author

andywenk commented Nov 15, 2017

@Bramjetten oh cool - ActiveStorage is nice. Thanks for the merge. It would be cool to have a section in the README about how to use S3. If I find a time slot I will send another PR.

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.

3 participants