Skip to content

Conversation

@nsdeschenes
Copy link
Contributor

This PR updates the Group By dropdown to support searching attributes via the API dealing with the 3000 attribute limit issue.

@nsdeschenes nsdeschenes requested a review from Zylphrex November 25, 2025 20:05
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 25, 2025
@nsdeschenes
Copy link
Contributor Author

@sentry review

@nsdeschenes nsdeschenes marked this pull request as ready for review November 26, 2025 15:18
@nsdeschenes nsdeschenes requested a review from a team as a code owner November 26, 2025 15:18
enabled,
queryKey,
queryFn: () => getTraceItemAttributeKeys(),
queryKey: [...queryKey, search],
Copy link
Member

Choose a reason for hiding this comment

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

You should pass the search to makeTraceItemAttributeKeysQueryOptions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That will cause an eslint violation 😢

@nsdeschenes nsdeschenes merged commit d40be40 into master Nov 26, 2025
47 of 48 checks passed
@nsdeschenes nsdeschenes deleted the nd/feat-explore-enabling-searching-attributes-on-group-by branch November 26, 2025 16:28
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.

3 participants