Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in

Delta Between Two Patch Sets: templates/all.html

Issue 5373080: Make /all?closed=1 param in URL show only closed issues (Closed)
Left Patch Set: Also fix pagination header Created 4 years, 5 months ago
Right Patch Set: Normalize 'closed' param and limit to known values Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
1 {%extends "issue_pagination.html"%} 1 {%extends "issue_pagination.html"%}
2 {%block subtitle%}Recent Issues 2 {%block subtitle%}Recent Issues
3 {%if closed%} 3 {%if closed|lower == 'false' %} (opened only) {%endif%}
4 {%if closed == '0' or closed == 'false' %} 4 {%if closed|lower == 'true' %} (closed only) {%endif%}
Andi Albrecht 2011/11/14 13:37:27 IMO we should unify this in the views.py when vali
techtonik 2011/11/14 17:13:21 Done. I've run into two 'interesting' bugs while
5 (opened only)
6 {%else%}
7 (closed only)
8 {%endif%}
9 {%endif%}
10 {%endblock%} 5 {%endblock%}

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 80a51fa-tainted