-
Notifications
You must be signed in to change notification settings - Fork 53
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
ImageWidget and Amazon S3 #3
Comments
I should really have a |
Thanks - I've just figured and out and came to respond, and you had already replied (that was quick!)... I'll prepare a pull request. |
Apparently, once PIL is installed, all tests pass. Is it safe to post a pull request? |
sure! ideally your pull request would also add a new test for the thing it is fixing, but I can take a look, it might be ok to merge without that if it passes existing tests. |
Hey, been using this in another project (using S3 again), and it seems to be working well. Seeing the S3-hosted-image. |
Correction: Still not solved. |
So, the two don't play together when you don't use utilize any thumbnail generator.
The reason is that instead of invoking the ImageField's url method, it performs a "manual" join between the project's media folder and the image path - and that simply doesn't work with remote storages such as Amazon's.
I have forked and fixed it for myself, and would like to send a pull request, but I have can't figure out how to run the tests - for some reason they are in the egg, but not in the app itself...
How do I run the tests?
The text was updated successfully, but these errors were encountered: