Skip to content

ref: remove daily-summary feature#110181

Open
wedamija wants to merge 3 commits intomasterfrom
flag-burner/rollout/remove-daily-summary
Open

ref: remove daily-summary feature#110181
wedamija wants to merge 3 commits intomasterfrom
flag-burner/rollout/remove-daily-summary

Conversation

@wedamija
Copy link
Member

@wedamija wedamija commented Mar 7, 2026

The daily-summary feature was only enabled for Sentry-internal and ~100 allowlisted orgs (la=1.0, no ea/ga rollout). Tests were mostly skipped with comments like "flaky and part of a dead project".

The daily-summary feature was only enabled for Sentry-internal and
~100 allowlisted orgs (la=1.0, no ea/ga rollout). Tests were mostly
skipped with comments like "flaky and part of a dead project".

Remove:
- Task: daily_summary.py
- Notification: DailySummaryNotification
- Slack builder: SlackDailySummaryMessageBuilder
- 6 Snuba referrer constants
- DailySummaryProjectContext and check_if_ctx_is_empty from utils
- Feature flag registration
@wedamija wedamija requested a review from ceorourke March 7, 2026 00:34
@wedamija wedamija requested review from a team as code owners March 7, 2026 00:34
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 7, 2026
cursor[bot]

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant