feat: add links to response of GET /api/club/{club_id}

This commit is contained in:
imperosol
2026-05-02 19:31:03 +02:00
parent 59847b3973
commit b9b0c00b74
3 changed files with 9 additions and 3 deletions
+1 -1
View File
@@ -41,7 +41,7 @@ class ClubController(ControllerBase):
queryset=Membership.objects.ongoing().select_related("user", "role"), queryset=Membership.objects.ongoing().select_related("user", "role"),
) )
return self.get_object_or_exception( return self.get_object_or_exception(
Club.objects.prefetch_related(prefetch), id=club_id Club.objects.prefetch_related(prefetch, "links"), id=club_id
) )
+6
View File
@@ -2,6 +2,7 @@ from typing import Annotated
from django.db.models import Q from django.db.models import Q
from ninja import FilterLookup, FilterSchema, ModelSchema from ninja import FilterLookup, FilterSchema, ModelSchema
from pydantic import HttpUrl
from club.models import Club, ClubRole, Membership from club.models import Club, ClubRole, Membership
from core.schemas import NonEmptyStr, SimpleUserSchema from core.schemas import NonEmptyStr, SimpleUserSchema
@@ -62,6 +63,11 @@ class ClubSchema(ModelSchema):
fields = ["id", "name", "logo", "is_active", "short_description", "address"] fields = ["id", "name", "logo", "is_active", "short_description", "address"]
members: list[ClubMemberSchema] members: list[ClubMemberSchema]
links: list[HttpUrl]
@staticmethod
def resolve_links(obj: Club):
return [link.url for link in obj.links.all()]
class UserMembershipSchema(ModelSchema): class UserMembershipSchema(ModelSchema):
+2 -2
View File
@@ -88,8 +88,8 @@ class TestFetchClub:
def test_fetch_club_nb_queries(self, client: Client, club: Club): def test_fetch_club_nb_queries(self, client: Client, club: Club):
user = subscriber_user.make() user = subscriber_user.make()
client.force_login(user) client.force_login(user)
with assertNumQueries(6): with assertNumQueries(7):
# - 4 queries for authentication # - 4 queries for authentication
# - 2 queries for the actual data # - 3 queries for the actual data
res = client.get(reverse("api:fetch_club", kwargs={"club_id": club.id})) res = client.get(reverse("api:fetch_club", kwargs={"club_id": club.id}))
assert res.status_code == 200 assert res.status_code == 200