Skip to content

feat(weekly-report): Total Issues chart rendering#119229

Merged
amy-chen23 merged 5 commits into
masterfrom
amyc/weekly-report-issues-chart-rendering
Jul 9, 2026
Merged

feat(weekly-report): Total Issues chart rendering#119229
amy-chen23 merged 5 commits into
masterfrom
amyc/weekly-report-issues-chart-rendering

Conversation

@amy-chen23

@amy-chen23 amy-chen23 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Resolves frontend rendering for ID-1669

Relies on data layer PR

Goal: add Total Project Issues graph to replace the "Top Transactions graph"

To test:

  • Start dev environment
  • Go to debug/mail/weekly-reports/?seed=90

To note

  • Gated behind weekly-report-issues-chart flag, but not sure if it's needed
  • Debug preview defaults to FF on, but go to debug/mail/weekly-reports/?show_issues_chart=0&seed=90 to see what the email looks like when the FF is off

Before:
image

After:
image

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 8, 2026
@amy-chen23 amy-chen23 changed the title feat(weekly-report): Issues chart rendering feat(weekly-report): Total Issues chart rendering Jul 8, 2026
@amy-chen23 amy-chen23 force-pushed the amyc/weekly-report-issues-chart-rendering branch from abe29d3 to e997a1c Compare July 9, 2026 18:51
@amy-chen23 amy-chen23 force-pushed the amyc/weekly-report-issues-chart-rendering branch from e997a1c to c6562a9 Compare July 9, 2026 19:00
@amy-chen23 amy-chen23 marked this pull request as ready for review July 9, 2026 20:22
@amy-chen23 amy-chen23 requested review from a team as code owners July 9, 2026 20:22
@amy-chen23 amy-chen23 requested a review from a team July 9, 2026 20:22

@shashjar shashjar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, as discussed i think we can remove the FF and just ship this globally

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 66488ca. Configure here.

Comment thread src/sentry/templates/sentry/emails/reports/body.html
Comment thread src/sentry/templates/sentry/emails/reports/body.html
@amy-chen23 amy-chen23 merged commit e1799f9 into master Jul 9, 2026
65 checks passed
@amy-chen23 amy-chen23 deleted the amyc/weekly-report-issues-chart-rendering branch July 9, 2026 22:18
@linear-code

linear-code Bot commented Jul 9, 2026

Copy link
Copy Markdown

ID-1669

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.

3 participants