(PC-31759)[API] feat: add params to log offer list view in adage

pass-culture · Sep 20, 2024 · daa12c0 · daa12c0
1 parent c285eec
commit daa12c0
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 1 deletion.
diff --git a/api/src/pcapi/routes/adage_iframe/logs.py b/api/src/pcapi/routes/adage_iframe/logs.py
@@ -48,6 +48,7 @@ def log_offer_list_view_switch(
             "source": body.source,
             "from": body.iframeFrom,
             "queryId": body.queryId,
+            "isMobile": body.isMobile,
         },
         uai=authenticated_information.uai,
         user_role=AdageFrontRoles.REDACTOR if institution else AdageFrontRoles.READONLY,

diff --git a/api/src/pcapi/routes/adage_iframe/serialization/logs.py b/api/src/pcapi/routes/adage_iframe/serialization/logs.py
@@ -21,6 +21,7 @@ class AdagePlaylistType(enum.Enum):
 
 class OfferListSwitch(AdageBaseModel):
     source: str
+    isMobile: bool | None
 
 
 class CatalogViewBody(AdageBaseModel):

diff --git a/api/tests/routes/adage_iframe/post_logs_test.py b/api/tests/routes/adage_iframe/post_logs_test.py
@@ -42,7 +42,7 @@ def test_log_offer_list_view_switch(self, test_client, caplog):
         with caplog.at_level(logging.INFO):
             response = test_client.post(
                 url_for("adage_iframe.log_offer_list_view_switch"),
-                json={"source": "offer_switch", "iframeFrom": "playlist", "queryId": "1234"},
+                json={"source": "offer_switch", "iframeFrom": "playlist", "queryId": "1234", "isMobile": True},
             )
 
         assert response.status_code == 204
@@ -51,6 +51,7 @@ def test_log_offer_list_view_switch(self, test_client, caplog):
             "analyticsSource": "adage",
             "source": "offer_switch",
             "queryId": "1234",
+            "isMobile": True,
             "from": "playlist",
             "uai": UAI,
             "user_role": AdageFrontRoles.READONLY,

diff --git a/pro/src/apiClient/adage/models/OfferListSwitch.ts b/pro/src/apiClient/adage/models/OfferListSwitch.ts
@@ -5,6 +5,7 @@
 export type OfferListSwitch = {
   iframeFrom: string;
   isFromNoResult?: boolean | null;
+  isMobile?: boolean | null;
   queryId?: string | null;
   source: string;
 };