Skip to content

[DataGridPremium] Fix aggregated values sorting #17326

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Apr 14, 2025
Merged

[DataGridPremium] Fix aggregated values sorting #17326

merged 2 commits into from
Apr 14, 2025

Conversation

cherniavskii
Copy link
Member

@cherniavskii cherniavskii commented Apr 11, 2025 •
edited
Loading

@cherniavskii cherniavskii added component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse plan: Premium Impact at least one Premium user feature: Sorting Related to the data grid Sorting feature feature: Aggregation Related to the data grid Aggregation feature labels Apr 11, 2025
@mui-bot
Copy link

mui-bot commented Apr 11, 2025

Deploy preview: https://deploy-preview-17326--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 9c589b9

@cherniavskii cherniavskii marked this pull request as ready for review April 11, 2025 14:49
@cherniavskii cherniavskii merged commit dfb2448 into mui:master Apr 14, 2025
31 checks passed
@cherniavskii cherniavskii deleted the fix-broken-sorting branch April 14, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arminmeh arminmeh arminmeh approved these changes

Assignees
No one assigned
Labels
component: data grid This is the name of the generic UI component, not the React module! feature: Aggregation Related to the data grid Aggregation feature feature: Sorting Related to the data grid Sorting feature plan: Premium Impact at least one Premium user regression A bug, but worse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cherniavskii @mui-bot @arminmeh