Fixed #18450 -- Removed default values for i18n JavaScript URLs in Admin templates #139

Closed
wants to merge 1 commit into from

3 participants

@jphalip
Django member

Although I'm not sure why right now, I wonder if the 'default' filter is there for a reason. Why do you think it's safe for it to be removed?

@apollo13
Django member

Cause newstyle urlresolving should always work nowadays. I think this is from the time where we had to support old admin urls which where just relative and the namespace refactor.

@apollo13 apollo13 closed this Jun 18, 2012
@nanuxbe nanuxbe pushed a commit to nanuxbe/django that referenced this pull request Jul 2, 2016
@bmispelon bmispelon Fixed #139 -- Replaced all tabs in templates with two spaces.
Command used:
    git grep -Il '      ' -- templates | xargs sed -i 's/       /  /g'
1fe4130
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment