fix(tracemetrics): Fix analytics firing (#103982)

k-fish · web-flow · commit 2928baa2c3df · 2025-11-25T17:46:42.000Z
### Summary
Metadata panel should fire and not be based on pending.

Closes LOGS-517
diff --git a/static/app/views/explore/hooks/useAnalytics.tsx b/static/app/views/explore/hooks/useAnalytics.tsx
@@ -740,7 +740,7 @@ export function useMetricsPanelAnalytics({
       return;
     }
 
-    if (metricSamplesTableResult.result.isFetching || !metricTimeseriesResult.isPending) {
+    if (metricSamplesTableResult.result.isFetching || metricTimeseriesResult.isPending) {
       return;
     }
 
@@ -810,7 +810,7 @@ export function useMetricsPanelAnalytics({
 
     if (
       metricAggregatesTableResult.result.isPending ||
-      !metricTimeseriesResult.isPending
+      metricTimeseriesResult.isPending
     ) {
       return;
     }

Original file line number	Diff line number	Diff line change
`@@ -740,7 +740,7 @@ export function useMetricsPanelAnalytics({`
`740`	`740`	`return;`
`741`	`741`	`}`
`742`	`742`
`743`		`- if (metricSamplesTableResult.result.isFetching \|\| !metricTimeseriesResult.isPending) {`
	`743`	`+ if (metricSamplesTableResult.result.isFetching \|\| metricTimeseriesResult.isPending) {`
`744`	`744`	`return;`
`745`	`745`	`}`
`746`	`746`
`@@ -810,7 +810,7 @@ export function useMetricsPanelAnalytics({`
`810`	`810`
`811`	`811`	`if (`
`812`	`812`	`metricAggregatesTableResult.result.isPending \|\|`
`813`		`- !metricTimeseriesResult.isPending`
	`813`	`+ metricTimeseriesResult.isPending`
`814`	`814`	`) {`
`815`	`815`	`return;`
`816`	`816`	`}`