Skip to content
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

[DataGridPremium] Fix error when aggregated column is removed #16181

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[DataGridPremium] Fix error when aggregated column is removed #16181

wants to merge 1 commit into from
+2 −1

Conversation

cherniavskii
Copy link
Member

@cherniavskii cherniavskii commented Jan 14, 2025 •
edited
Loading

Fixes #16091

@cherniavskii cherniavskii added bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! plan: Premium Impact at least one Premium user feature: Aggregation Related to the data grid Aggregation feature labels Jan 14, 2025
@mui-bot
Copy link

mui-bot commented Jan 14, 2025

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

Generated by 🚫 dangerJS against a1303d2

@lauri865
Copy link
Contributor

lauri865 commented Jan 15, 2025 •
edited
Loading

#15627 also fixes this and any derivative of the same root cause. Could be cherry picked probably if we ignore the detail panel changes. Happy to make it merge-ready again, but would prefer #15623 to get merged first to avoid any extra work.

@lauri865 lauri865 mentioned this pull request Jan 21, 2025
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug 🐛 Something doesn't work 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 plan: Premium Impact at least one Premium user
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[data grid] Removing column with turned on aggregation leads to crash
3 participants
@cherniavskii @mui-bot @lauri865