Fix trackpageview

elastic · Oct 29, 2020 · 963fd72 · 963fd72
1 parent 0c832f4
commit 963fd72
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/x-pack/plugins/apm/public/components/app/service_inventory/index.tsx b/x-pack/plugins/apm/public/components/app/service_inventory/index.tsx
@@ -86,11 +86,12 @@ export function ServiceInventory() {
     }
   }, [data.hasLegacyData, core.http.basePath, core.notifications.toasts]);
 
-  // Tracking both of these to keep our data backwards compatible
+  // The page is called "service inventory" to avoid confusion with the
+  // "service overview", but this is tracked in some dashboards because it's the
+  // initial landing page for APM, so it stays as "services_overview" (plural.)
+  // for backward compatibility.
   useTrackPageview({ app: 'apm', path: 'services_overview' });
-  useTrackPageview({ app: 'apm', path: 'service_inventory' });
   useTrackPageview({ app: 'apm', path: 'services_overview', delay: 15000 });
-  useTrackPageview({ app: 'apm', path: 'service_inventory', delay: 15000 });
 
   const localFiltersConfig: React.ComponentProps<typeof LocalUIFilters> = useMemo(
     () => ({