Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Insights: clicking on transaction on domain overview page populates search filters on transaction page #82661

Open
bcoe opened this issue Dec 30, 2024 · 1 comment

Comments

@bcoe
Copy link
Member

bcoe commented Dec 30, 2024

Image

Could we just do something like this?

diff --git a/static/app/views/performance/table.tsx b/static/app/views/performance/table.tsx
index 15fed2e06b4..57e26bf2457 100644
--- a/static/app/views/performance/table.tsx
+++ b/static/app/views/performance/table.tsx
@@ -283,6 +283,10 @@ class _Table extends Component<Props, State> {
       }
       const {isInDomainView, view} = this.props.domainViewFilters ?? {};
 
+      const query = summaryView.generateQueryStringObject();
+      if (dataRow.hasOwnProperty('transaction.op')) {
+        query.query = `( transaction.op:${dataRow['transaction.op'].toString()} )`;
+      }
       const target = isUnparameterizedRow
         ? createUnnamedTransactionsDiscoverTarget({
             organization,
@@ -291,7 +295,7 @@ class _Table extends Component<Props, State> {
         : transactionSummaryRouteWithQuery({
             orgSlug: organization.slug,
             transaction: String(dataRow.transaction) || '',
-            query: summaryView.generateQueryStringObject(),
+            query,
             projectID,
             view: (isInDomainView && view) || undefined,
           });
@getsantry
Copy link
Contributor

getsantry bot commented Dec 30, 2024

Routing to @getsentry/product-owners-insights for triage ⏲️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

2 participants