Skip to content

fix(insights): Fix empty txn summary when transaction.op is default#113099

Merged
mjq merged 1 commit intomasterfrom
mjq/fix-default-transaction-op
Apr 15, 2026
Merged

fix(insights): Fix empty txn summary when transaction.op is default#113099
mjq merged 1 commit intomasterfrom
mjq/fix-default-transaction-op

Conversation

@mjq
Copy link
Copy Markdown
Member

@mjq mjq commented Apr 15, 2026

Relay doesn't extract transaction ops when they have the default value of default. Since links into the transaction summary from insights include the transaction op as a filter, transaction.op:default was returning an empty screen.

Instead, use span.op when linking into transaction summary:

  • default is populated here correctly, so the page works
  • it's span first/span streaming compatible

This fixes both the Insights and Prebuilt Dashboard versions of the transaction link rendering (and removes an old comment calling out the span.op/transaction.op name discrepancy).

…efault

Relay doesn't extract transaction ops when they have the default value of `default`. Since links into the transaction summary from insights include the transaction op as a filter, `transaction.op:default` was returning an empty screen.

Instead, use `span.op`:
- `default` is populated here correctly, so the page works
- it's span first/span streaming compatible
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 15, 2026
@mjq mjq marked this pull request as ready for review April 15, 2026 20:08
@mjq mjq requested a review from a team as a code owner April 15, 2026 20:08
@mjq mjq enabled auto-merge (squash) April 15, 2026 20:12
@mjq mjq merged commit 2d433b2 into master Apr 15, 2026
74 of 76 checks passed
@mjq mjq deleted the mjq/fix-default-transaction-op branch April 15, 2026 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants