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

ImageWidget and Amazon S3 #3

Open
danielbraun opened this issue Jun 5, 2013 · 6 comments
Open

ImageWidget and Amazon S3 #3

danielbraun opened this issue Jun 5, 2013 · 6 comments

Comments

@danielbraun
Copy link

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?

@carljm
Copy link
Owner

carljm commented Jun 5, 2013

python setup.py test will run the tests using your current environment (requires that you have Django installed, and probably PIL too if you want the ImageWidget tests to run). Alternatively, you can run tox (if you "pip install tox") and it will run the tests against a variety of Python and Django versions (it may have some errors if you don't have all the requisite Python versions available).

I should really have a CONTRIBUTING doc...

@danielbraun
Copy link
Author

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.

@danielbraun
Copy link
Author

Apparently, once PIL is installed, all tests pass. Is it safe to post a pull request?

@carljm
Copy link
Owner

carljm commented Jun 5, 2013

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.

@danielbraun
Copy link
Author

Hey, been using this in another project (using S3 again), and it seems to be working well. Seeing the S3-hosted-image.
Closing.

@danielbraun
Copy link
Author

Correction: Still not solved.

@danielbraun danielbraun reopened this Nov 30, 2014
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

No branches or pull requests

2 participants