Skip to content

fix(slack): Include y-axis aggregate in Explore unfurl title#112693

Merged
DominikB2014 merged 2 commits intomasterfrom
dominikbuszowiecki/dain-1489-generated-title-should-match-aggregate
Apr 10, 2026
Merged

fix(slack): Include y-axis aggregate in Explore unfurl title#112693
DominikB2014 merged 2 commits intomasterfrom
dominikbuszowiecki/dain-1489-generated-title-should-match-aggregate

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

The Explore unfurl title was a generic "Explore Traces" or "Explore Logs"
which didn't tell the user what was actually being plotted. Now the title
includes the aggregate, e.g. "Explore Traces - avg(span.duration)" or
"Explore Logs - count(message)", matching what the user sees in the Explore
page.

Fixes DAIN-1489

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 10, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 10, 2026
@DominikB2014 DominikB2014 requested a review from a team April 10, 2026 17:54
Add Slack unfurl support for Explore Logs URLs (`/explore/logs/`),
alongside the
existing Explore Traces URL support.

The handler detects the URL path to determine logs vs traces, then sets
the
correct dataset (`ourlogs` for logs, `spans` for traces) and title
(`Explore Logs` vs `Explore Traces`). The existing
`map_explore_query_args`
already supported the `aggregateField` param format used by logs URLs,
so only
the URL matching, dataset selection, and title needed to change.

Supports both `/organizations/{slug}/explore/logs/` and customer domain
`{slug}.sentry.io/explore/logs/` URL formats.

Fixes DAIN-1478

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@DominikB2014 DominikB2014 force-pushed the dominikbuszowiecki/dain-1489-generated-title-should-match-aggregate branch from 9a635f0 to 6f820d9 Compare April 10, 2026 17:54
The unfurl title now includes the aggregates being plotted, e.g.
"Explore Traces - avg(span.duration)" or "Explore Logs - count(message)"
instead of just "Explore Traces". This matches the Explore page and
helps users understand what's being charted at a glance.

Fixes DAIN-1489
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@DominikB2014 DominikB2014 force-pushed the dominikbuszowiecki/dain-1489-generated-title-should-match-aggregate branch from 6f820d9 to 5bcf510 Compare April 10, 2026 17:55
@DominikB2014 DominikB2014 marked this pull request as ready for review April 10, 2026 17:57
@DominikB2014 DominikB2014 requested review from a team as code owners April 10, 2026 17:57
@DominikB2014 DominikB2014 enabled auto-merge (squash) April 10, 2026 18:02
@DominikB2014 DominikB2014 merged commit 7820fdf into master Apr 10, 2026
56 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1489-generated-title-should-match-aggregate branch April 10, 2026 18:06
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.

2 participants