From f88c0307ea61bb66265e211c074ce6e6f3c98f48 Mon Sep 17 00:00:00 2001 From: Florent Chehab Date: Tue, 7 Jul 2026 15:32:47 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B(summary)=20do=20not=20save=20null?= =?UTF-8?q?=20emails=20in=20analytics?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In case the email is not provided we should not update the analytics email value. --- src/summary/summary/api/route/tasks_v2.py | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/src/summary/summary/api/route/tasks_v2.py b/src/summary/summary/api/route/tasks_v2.py index ec175160..1edd10fd 100644 --- a/src/summary/summary/api/route/tasks_v2.py +++ b/src/summary/summary/api/route/tasks_v2.py @@ -59,16 +59,16 @@ async def create_transcribe_task_v2( ] ) + properties = {} + if request.user_email: + properties["$set"] = {"email": request.user_email} + # We track the request, this also properly initializes the user in the # analytics system, so that later feature flags work properly analytics.capture( settings.posthog_transcript_request, request.user_sub, - properties={ - "$set": { - "email": request.user_email, - }, - }, + properties=properties, ) return TranscribeWebhookPendingPayload(job_id=task.id).model_dump() @@ -92,14 +92,15 @@ async def create_summarize_task_v2( # We track the request, this also properly initializes the user in the # analytics system, so that later feature flags work properly + + properties = {} + if request.user_email: + properties["$set"] = {"email": request.user_email} + analytics.capture( settings.posthog_summary_request, request.user_sub, - properties={ - "$set": { - "email": request.user_email, - }, - }, + properties=properties, ) return SummarizeWebhookPendingPayload(job_id=task.id).model_dump()