mirror of
https://github.com/suitenumerique/meet.git
synced 2026-07-26 11:58:53 +00:00
🔥(summary) remove call to summary enabled feature flag
We now rely only on the value coming from the API request. This feature flag should be handled by the caller.
This commit is contained in:
@@ -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
|
||||
|
||||
|
||||
@@ -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
|
||||
|
||||
|
||||
Reference in New Issue
Block a user