Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

url inconsistency #147

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants

fuu commented Jun 9, 2012

according to admin.site.urls, there should be a tailing '/'.

@apollo13 apollo13 commented on the diff Jun 9, 2012

django/contrib/admin/templates/admin/object_history.html
@@ -7,7 +7,7 @@
<a href="{% url 'admin:index' %}">{% trans 'Home' %}</a>
&rsaquo; <a href="{% url 'admin:app_list' app_label=app_label %}">{{ app_label|capfirst|escape }}</a>
&rsaquo; <a href="{% url opts|admin_urlname:'changelist' %}">{{ module_name }}</a>
-&rsaquo; <a href="{% url opts|admin_urlname:'changelist' %}{{ object.pk }}">{{ object|truncatewords:"18" }}</a>
+&rsaquo; <a href="{% url opts|admin_urlname:'changelist' %}{{ object.pk }}/">{{ object|truncatewords:"18" }}</a>
@apollo13

apollo13 Jun 9, 2012

Owner

Shouldn't this be {% url opts|admin_urlname:'change' object.pk %} instead?

Member

spookylukey commented Jun 10, 2012

apollo13's correct fix has been applied in e9497a3

Owner

jezdez commented Jun 10, 2012

Awesome, thanks!

@sztrovacsek sztrovacsek pushed a commit to sztrovacsek/django that referenced this pull request Mar 7, 2015

@evildmp evildmp Merge pull request #147 from VincentHussey/VincentHussey
Update attendees_and_learners.rst
9c0152c

@nanuxbe nanuxbe pushed a commit to nanuxbe/django that referenced this pull request Jul 2, 2016

@olasitarska olasitarska Copy pasted styles for bars from current version Refs #147 e71f735
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment