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

use staticfiles finders to find media files if STATIC_URL is defined #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

derek73
Copy link

@derek73 derek73 commented Feb 5, 2011

Django 1.3 introduces staticfiles to store static app-specific media files. The files can be located in various locations around your project that you can define through your STATICFILES_FINDERS and STATICFILES_DIRS settings. This change uses the staticfiles finder to find the media files.

@monokrome
Copy link

In my opinion, you shouldn't be doing this in production, because you should be using collectstatic and serving static files through your HTTP server or a static file service and not through Django itself. Not only is efficiency an issue, but this patch would only provide unneeded extra confusion to developers in debug mode - so, I believe that it is not helpful in production or debug environments.

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.

2 participants