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

After upgrading to CarrierWave 1.0+, need to reload model for file not to be nil #303

Open
cgriego opened this issue May 8, 2018 · 10 comments
Labels

Comments

@cgriego
Copy link

cgriego commented May 8, 2018

After upgrading CarrierWave to 1.0.0.beta or higher, tests using Cloudinary + CarrierWave started failing because model.image.file were nil until the model was reloaded.

I think this person ran into the same issue.
https://stackoverflow.com/questions/46271081/carrierwave-cloudinary-image-upload-works-but-null-values-returned-right-after-u

@shirlymanor
Copy link

I apologize for the very late reply.
Unfortunately, it's a known bug in carriwave (there are a couple of open issues on GH). A workaround will be to reload as you mention.

@bbugh
Copy link

bbugh commented Oct 23, 2018

@cgriego did you make any progress with this other than forcing the model to reload? I can't find any issues in carrierwave referencing this either. Thanks!

@cgriego
Copy link
Author

cgriego commented Oct 23, 2018

I still have the model reload workaround in place.

@shirlymanor
Copy link

Closing this issue due to the time elapsed. Please feel free to either re-open the issue, contact our support at http://support.cloudinary.com or create a new ticket if you have any additional issues.

@cgriego
Copy link
Author

cgriego commented May 13, 2019

I can confirm this issue is not happening with carrierwave 1.3.1 and cloudinary 1.11.1.

@cgriego
Copy link
Author

cgriego commented May 13, 2019

Sorry, yes this issue does still happen with the latest gem versions mentioned above. Please reopen.

@shirlymanor shirlymanor assigned tocker and unassigned shirlymanor May 13, 2019
@aceunreal
Copy link

Could this be re-opened?

@momoip
Copy link
Collaborator

momoip commented Apr 12, 2021

Hi,
I have raised an internal ticket and our Dev team will be looking at this issue.
Thanks,
Mo

@david-zw-liu
Copy link
Contributor

david-zw-liu commented Mar 25, 2022

Any update?

# same here

gem 'carrierwave', '~> 2.2.2'
gem 'cloudinary', '~> 1.22.0'

@aleksandar-cloudinary
Copy link
Contributor

Hi, @david-liu-950627 - Thanks for following up on this and for creating the PR. I have added one of my colleagues as a reviewer.

@tocker tocker removed their assignment Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

10 participants