From b927be9f16dee891df2cd8665ee48c353974fd38 Mon Sep 17 00:00:00 2001 From: lebaudantoine Date: Thu, 17 Apr 2025 15:31:35 +0200 Subject: [PATCH] =?UTF-8?q?=E2=9C=A8(frontend)=20serialize=20recording=20k?= =?UTF-8?q?ey=20for=20frontend=20download=20links?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add recording key to serialized API response to enable frontend to generate proper download links without additional backend calls. Simplifies media access workflow across the application. --- src/backend/core/api/serializers.py | 2 +- src/backend/core/tests/recording/test_api_recordings_list.py | 1 + .../core/tests/recording/test_api_recordings_retrieve.py | 2 ++ 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/backend/core/api/serializers.py b/src/backend/core/api/serializers.py index b7065da7..2e8ae574 100644 --- a/src/backend/core/api/serializers.py +++ b/src/backend/core/api/serializers.py @@ -156,7 +156,7 @@ class RecordingSerializer(serializers.ModelSerializer): class Meta: model = models.Recording - fields = ["id", "room", "created_at", "updated_at", "status", "mode"] + fields = ["id", "room", "created_at", "updated_at", "status", "mode", "key"] read_only_fields = fields diff --git a/src/backend/core/tests/recording/test_api_recordings_list.py b/src/backend/core/tests/recording/test_api_recordings_list.py index e22fa360..44bac4f9 100644 --- a/src/backend/core/tests/recording/test_api_recordings_list.py +++ b/src/backend/core/tests/recording/test_api_recordings_list.py @@ -78,6 +78,7 @@ def test_api_recordings_list_authenticated_direct(role): assert expected_ids == result_ids assert results[0] == { "id": str(recording.id), + "key": recording.key, "created_at": recording.created_at.isoformat().replace("+00:00", "Z"), "mode": recording.mode, "room": { diff --git a/src/backend/core/tests/recording/test_api_recordings_retrieve.py b/src/backend/core/tests/recording/test_api_recordings_retrieve.py index dca425f2..5ad84c62 100644 --- a/src/backend/core/tests/recording/test_api_recordings_retrieve.py +++ b/src/backend/core/tests/recording/test_api_recordings_retrieve.py @@ -80,6 +80,7 @@ def test_api_recording_retrieve_administrators(): assert content == { "id": str(recording.id), + "key": recording.key, "room": { "access_level": str(room.access_level), "id": str(room.id), @@ -111,6 +112,7 @@ def test_api_recording_retrieve_owners(): assert content == { "id": str(recording.id), + "key": recording.key, "room": { "access_level": str(room.access_level), "id": str(room.id),