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

Compatibility problem with carrierwave 0.6.0 #35

Merged
merged 3 commits into from Apr 13, 2012
Merged

Conversation

rmm5t
Copy link
Member

@rmm5t rmm5t commented Apr 8, 2012

Upgraded to carrierwave 0.6.0, and whenever I start the app, it says undefined method 'grid_fs_database=' for CarrierWave::Uploader::Base:Class)

@ahawkins
Copy link

+1

@yfeldblum
Copy link

Step 1: Patch carrierwave to have a list of registered drivers and to loop through the registered drivers and add config items that the registered drivers expose.

Step 2: Patch carrierwave-mongoid to register itself as a driver with carrierwave.

@rmm5t
Copy link
Member

rmm5t commented Apr 8, 2012

I started a carrierwave-0.6-support branch. Right now, all the existing specs are passing, but I haven't yet combed over everything to make sure it's stable. If anyone has time to test this branch on their projects in development in the meantime, that'd be a big help.

gem "carrierwave-mongoid", :git => "git://github.com/jnicklas/carrierwave-mongoid.git", :branch => "carrierwave-0.6-support"

@gkop
Copy link

gkop commented Apr 9, 2012

@rmm5t looks good here! It passed my app tests for https://github.com/gkop/bbase and is working fine in production with my legacy dataset as well as new uploads. This is on an upgrade from carrierwave 0.5.4, with the latest stable mongo and mongoid. Thanks!

@jtescher
Copy link

Works great for me! mongoid-2.4.8, carrierwave-0.6.1

@benzheren
Copy link
Author

This branch works for me as well with mongoid-2.4.8 and carrierwave-0.6.1

@jhirn
Copy link

jhirn commented Apr 11, 2012

+1

@rmm5t rmm5t merged commit 8f8a918 into master Apr 13, 2012
@rmm5t
Copy link
Member

rmm5t commented Apr 13, 2012

Thanks for the help testing, everyone. This is now in the master branch. I want to look into pulling in #34 before a 0.2.0 release, but that release should be soon.

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.

None yet

7 participants