From e6f0ff511207de968324ca408debd83ada7bf4ab Mon Sep 17 00:00:00 2001 From: Florent Chehab Date: Mon, 6 Jul 2026 21:45:29 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=94=A5(summary)=20remove=20call=20to=20su?= =?UTF-8?q?mmary=20enabled=20feature=20flag?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We now rely only on the value coming from the API request. This feature flag should be handled by the caller. --- CHANGELOG.md | 6 ++++++ src/summary/summary/core/celery_worker.py | 5 ----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 515e0237..7c0a0ca0 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,11 @@ and this project adheres to ## [Unreleased] +### Added + +- ✨(backend) extend analytics module to support feature flags +- ✨(backend) implement feature flags in Posthog analytics backend + ### Changed - ⬆️(agents) upgrade to python 3.14 slim @@ -15,6 +20,7 @@ and this project adheres to - 💥(summary) remove v1 related code #1362 - ✨(meet) use compatible with summary v2 #1362 - ♻️(backend) refactor analytics backend from Protocol to abstract class +- 🔥(summary) remove call to summary enabled feature flag ### Fixed diff --git a/src/summary/summary/core/celery_worker.py b/src/summary/summary/core/celery_worker.py index 091c5fae..867cbcf0 100644 --- a/src/summary/summary/core/celery_worker.py +++ b/src/summary/summary/core/celery_worker.py @@ -406,11 +406,6 @@ def _should_auto_create_summary(payload: TranscribeTaskJob) -> bool: reason = "Auto create summary is not requested in the payload" elif not settings.is_summary_enabled: reason = "Summary feature is disabled" - elif not analytics.is_feature_enabled( - "summary-enabled", - distinct_id=payload.user_sub, - ): - reason = "Summary feature flag return false" else: return True