Skip to content

Conversation

narsaynorath
Copy link
Member

Connect the group by options by making a request to trace-items for attributes under the specific metric name. Also adds in some quick UI cleanup by removing dummy metric options and debug info

I had to update the useTraceItemAttributeKeys to take a query prop and pass it along to the request.

Screen.Recording.2025-10-09.at.2.44.39.PM.mov

@narsaynorath narsaynorath requested a review from a team as a code owner October 9, 2025 18:50
Copy link

linear bot commented Oct 9, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 9, 2025
cursor[bot]

This comment was marked as outdated.

const groupBys = useQueryParamsGroupBys();
const setGroupBys = useSetQueryParamsGroupBys();

const metricNameFilter = metricName ? `metric.name:"${metricName}"` : undefined;
Copy link
Contributor

Choose a reason for hiding this comment

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

We have a brand new MutableSearch you could try out here 🥺

@narsaynorath narsaynorath merged commit 7d2afd7 into master Oct 9, 2025
45 checks passed
@narsaynorath narsaynorath deleted the narsaynorath/logs-404-show-applicable-group-by-options-in-select-field branch October 9, 2025 19:11
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