feat(explore): Add Chartcuterie config for Explore heat maps#118253
Merged
gggritso merged 3 commits intoJun 23, 2026
Merged
Conversation
github-actions
Bot
added
the
Scope: Frontend
Member
Author
|
@cursor review |
Contributor
Story previewsPreview the stories changed in this PR on the Vercel deployment: Preview deployment: https://sentry-1jdgo22db.sentry.dev |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9258c83. Configure here.
Contributor
📊 Type Coverage Diff✅ no issues found |
nsdeschenes
approved these changes
Jun 23, 2026
nsdeschenes
left a comment
nsdeschenes
left a comment
Contributor
There was a problem hiding this comment.
Looks good to me, just one comment that might make the PR typechecker thing happier 😄
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
gggritso
deleted the
georgegritsouk/logs-889-add-chartcuterie-configurations-for-heat-maps-in-explore
branch
gggritso
added a commit
that referenced
this pull request
Jun 23, 2026
The heatmaps chartcuterie config (added in #118253) calls formatXAxisTimestamp with the old {utc} object signature. Update to use the new string timezone parameter. Also re-apply timezone changes to the heat map visualization that were overwritten by the merge. Co-Authored-By: Claude <noreply@anthropic.com>
sehr-m
pushed a commit
that referenced
this pull request
Jun 23, 2026
To render Heat Map charts in Chartcuterie, Sentry must add a configuration for that type. That's what this PR does. Note that a lot of the configuration code is shared/similar between the Chartcuterie setup and `HeatMapWidgetVisualization` but that's intentional. The _palette_ is shared (important) but Chartcuterie configuration is different since it's always in UTC, it _never_ has any hover actions, and it has slightly different UI due to the server-rendered nature of it. **e.g.,** <img width="1200" height="400" alt="heat-map-sample" src="https://github.com/user-attachments/assets/3d2205ff-947d-4dcd-8717-d2c535e2cb57" />
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To render Heat Map charts in Chartcuterie, Sentry must add a configuration for that type. That's what this PR does.
Note that a lot of the configuration code is shared/similar between the Chartcuterie setup and
HeatMapWidgetVisualizationbut that's intentional. The palette is shared (important) but Chartcuterie configuration is different since it's always in UTC, it never has any hover actions, and it has slightly different UI due to the server-rendered nature of it.e.g.,
