Skip to content

fix(tracemetrics): Metric Selector should pass along environment filter#107440

Merged
narsaynorath merged 3 commits intomasterfrom
nar/fix/tracemetrics-metric-selector-should-scope-to-page-filters
Feb 3, 2026
Merged

fix(tracemetrics): Metric Selector should pass along environment filter#107440
narsaynorath merged 3 commits intomasterfrom
nar/fix/tracemetrics-metric-selector-should-scope-to-page-filters

Conversation

@narsaynorath
Copy link
Member

@narsaynorath narsaynorath commented Feb 2, 2026

The query for fetching metrics wasn't passing along the environment filter. This made it look like there were valid metrics when there were not.

Resolves JAVASCRIPT-36T1

The query for fetching metrics wasn't passing along the environment filter. This
made it look like there were valid metrics when there were not.
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 3, 2026
@narsaynorath narsaynorath enabled auto-merge (squash) February 3, 2026 14:54
@narsaynorath narsaynorath merged commit 7cfda37 into master Feb 3, 2026
59 checks passed
@narsaynorath narsaynorath deleted the nar/fix/tracemetrics-metric-selector-should-scope-to-page-filters branch February 3, 2026 14:55
jaydgoss pushed a commit that referenced this pull request Feb 12, 2026
…er (#107440)

The query for fetching metrics wasn't passing along the environment
filter. This made it look like there were valid metrics when there were
not.

Resolves JAVASCRIPT-36T1
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