From 9fdd5565a10f15d935cc9aee5e23cb612169808d Mon Sep 17 00:00:00 2001 From: imperosol Date: Wed, 13 May 2026 09:47:23 +0200 Subject: [PATCH 1/2] add og tags to club list page --- club/templates/club/club_list.jinja | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/club/templates/club/club_list.jinja b/club/templates/club/club_list.jinja index 8ae08bf3..19164143 100644 --- a/club/templates/club/club_list.jinja +++ b/club/templates/club/club_list.jinja @@ -1,6 +1,13 @@ {% if is_fragment %} {% extends "core/base_fragment.jinja" %} + {% block metatags %} + + + + + {% endblock %} + {# Don't display tabs and errors #} {% block tabs %} {% endblock %} From d3094a850fcc09bea9afb6d2dc5d319500dcd347 Mon Sep 17 00:00:00 2001 From: imperosol Date: Wed, 13 May 2026 10:12:52 +0200 Subject: [PATCH 2/2] fix: incorrect initial value for `ClubSearchForm.club_status` --- club/views.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/club/views.py b/club/views.py index cce60e8c..2df81a0e 100644 --- a/club/views.py +++ b/club/views.py @@ -209,15 +209,15 @@ class ClubListView(AllowFragment, FormMixin, ListView): def get_form_kwargs(self): res = super().get_form_kwargs() - if self.request.method == "GET": - res |= {"data": self.request.GET, "initial": self.request.GET} + # if request.GET is empty, the form will interpret club_status as None, + # even though we want it to be initially True, + # so we force a defaut True value. + res["data"] = {"club_status": True} | self.request.GET.dict() return res def get_queryset(self): form: ClubSearchForm = self.get_form() qs = self.queryset - if not form.is_bound: - return qs.filter(is_active=True) if not form.is_valid(): return qs.none() if name := form.cleaned_data.get("name"):