filter mobile transctions

getsentry · Oct 9, 2024 · 488a7a7 · 488a7a7
1 parent 29b24ac
commit 488a7a7
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/static/app/views/insights/pages/mobile/mobileOverviewPage.tsx b/static/app/views/insights/pages/mobile/mobileOverviewPage.tsx
@@ -24,6 +24,7 @@ import {ToolRibbon} from 'sentry/views/insights/common/components/ribbon';
 import {useOnboardingProject} from 'sentry/views/insights/common/queries/useOnboardingProject';
 import {ViewTrendsButton} from 'sentry/views/insights/common/viewTrendsButton';
 import {MobileHeader} from 'sentry/views/insights/pages/mobile/mobilePageHeader';
+import {OVERVIEW_PAGE_ALLOWED_OPS} from 'sentry/views/insights/pages/mobile/settings';
 import {OVERVIEW_PAGE_TITLE} from 'sentry/views/insights/pages/settings';
 import {
   generateGenericPerformanceEventView,
@@ -87,6 +88,12 @@ function MobileOverviewPage() {
     ? REACT_NATIVE_COLUMN_TITLES
     : MOBILE_COLUMN_TITLES;
 
+  const doubleChartRowEventView = eventView.clone(); // some of the double chart rows rely on span metrics, so they can't be queried the same way
+
+  const existingQuery = new MutableSearch(eventView.query);
+  existingQuery.addDisjunctionFilterValues('transaction.op', OVERVIEW_PAGE_ALLOWED_OPS);
+  eventView.query = existingQuery.formatString();
+
   const showOnboarding = onboardingProject !== undefined;
 
   const doubleChartRowCharts = [
@@ -193,7 +200,11 @@ function MobileOverviewPage() {
                 <PerformanceDisplayProvider
                   value={{performanceType: ProjectPerformanceType.MOBILE}}
                 >
-                  <DoubleChartRow allowedCharts={doubleChartRowCharts} {...sharedProps} />
+                  <DoubleChartRow
+                    allowedCharts={doubleChartRowCharts}
+                    {...sharedProps}
+                    eventView={doubleChartRowEventView}
+                  />
                   <TripleChartRow allowedCharts={tripleChartRowCharts} {...sharedProps} />
                   <Table
                     projects={projects}

diff --git a/static/app/views/insights/pages/mobile/settings.ts b/static/app/views/insights/pages/mobile/settings.ts
@@ -2,3 +2,14 @@ import {t} from 'sentry/locale';
 
 export const MOBILE_LANDING_SUB_PATH = 'mobile';
 export const MOBILE_LANDING_TITLE = t('Mobile');
+
+export const OVERVIEW_PAGE_ALLOWED_OPS = [
+  'ui.action.swipe',
+  'ui.action.scroll',
+  'ui.action.click',
+  'ui.load',
+  'app.lifecycle',
+  // navigation and pageload are seen in react-native
+  'navigation',
+  'pageload',
+];