Fix url template tags (and readme) for django 1.5 #6

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@meldra
meldra commented Apr 2, 2013

In Django 1.5 urls like {% url myurl %} must now be {% url 'myurl' %} or else django will spit out NoReverseMatch errors such as:

NoReverseMatch at /socialprofile/edit/
'url' requires a non-empty first argument. The syntax changed in Django 1.5, see the docs.

This patch converts the urls in the templates and adds {% load url from future %} for backwards compatibility with Django 1.3/1.4

Tested on django 1.4 and django 1.5

@cyface
Owner
cyface commented May 1, 2015

Merged in with the updates to the latest version! Thank you so much for the PR, and sorry for the delay in merging it in!

@cyface cyface closed this May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment