Skip to content

Commit 7212b4a

Browse files
committed
Quick fix for photo queue: reduce number of results shown
1 parent dad961f commit 7212b4a

3 files changed

Lines changed: 5 additions & 4 deletions

File tree

ynr/apps/moderation_queue/templates/moderation_queue/photo-review-list.html

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ <h1>Uploaded photos for review</h1>
5959
</tr>
6060
</thead>
6161
<tbody>
62-
{% for queued_image in object_list %}
62+
{% for queued_image in object_list.object_list %}
6363
<tr>
6464
<td>
6565
{% thumbnail queued_image.image "x100" as im %}
@@ -82,4 +82,3 @@ <h1>Uploaded photos for review</h1>
8282

8383

8484
{% endblock %}
85-

ynr/apps/moderation_queue/tests/test_queue.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ def test_photo_review_queue_view_logged_in_unprivileged(self):
160160

161161
def test_photo_review_queue_view_logged_in_privileged(self):
162162
queue_url = reverse("photo-review-list")
163-
with self.assertNumQueries(FuzzyInt(40, 41)):
163+
with self.assertNumQueries(FuzzyInt(40, 42)):
164164
response = self.app.get(queue_url, user=self.test_reviewer)
165165
self.assertEqual(response.status_code, 200)
166166
queue_table = response.html.find("table")

ynr/apps/moderation_queue/views.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
from urllib.parse import quote
55

66
import bleach
7+
from django.core.paginator import Paginator
8+
79
from auth_helpers.views import GroupRequiredMixin
810
from braces.views import LoginRequiredMixin
911
from candidates.models import TRUSTED_TO_LOCK_GROUP_NAME, Ballot, LoggedAction
@@ -129,7 +131,7 @@ def get_context_data(self, **kwargs: Any) -> Dict[str, Any]:
129131
request=self.request,
130132
)
131133
context["filter_obj"] = filter_obj
132-
context["object_list"] = filter_obj.qs
134+
context["object_list"] = Paginator(filter_obj.qs, 20).page(1)
133135
context["shortcuts"] = filter_obj.shortcuts
134136

135137
return context

0 commit comments

Comments
 (0)