Skip to content

Not sure if this is still maintained. Tiny fix to add STATIC_URL to the templatetag context. Thanks!#19

Open
rolo wants to merge 1 commit intoericflo:masterfrom
wildfish:master
Open

Not sure if this is still maintained. Tiny fix to add STATIC_URL to the templatetag context. Thanks!#19
rolo wants to merge 1 commit intoericflo:masterfrom
wildfish:master

Conversation

@rolo
Copy link
Copy Markdown

@rolo rolo commented Mar 29, 2011

Pass STATIC_URL to templatetag context as per new convention in Django 1.3

@jor123
Copy link
Copy Markdown

jor123 commented Jan 24, 2012

or 'keep' the whole context like in pull request #10

@hedleyroos
Copy link
Copy Markdown
Collaborator

STATIC_URL is already set by the django.core.context_processors.static context processor. You should add that to settings. Passing MEDIA_URL here is redundant because there is a context processor for that as well. Just adhering to DRY.

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.

3 participants