-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Data Grid Premium: Cannot read properties of null (reading 'type') at getCellAggregationResult #9859
Comments
Hey, @jon-spover thanks for reporting this. I may have missed something but looking at the reproduction you have provided your filter model is not correct. Instead of Regarding the error - I'm not seeing it in the example. Maybe you are using an older version? |
Hi @DanailH thanks for looking into this and fixing the confounding bug in my minimal reproduction. However, in your example I noticed the aggregation model is commented out. When I uncomment it (see fork here: https://codesandbox.io/s/long-snow-n4tphc), I again see the error: It seems there is an issue when the following 3 criteria are present:
|
Ah ok, I see the issue now. It does look like a valid bug. It looks like even if you don't set the |
Yes I experimented with trying to add a controlled filter and grouping model in the DataGrid and still ran into the same bug. Thanks for looking into this and let me know if there's anything I can do to help resolve! |
Same as #9876, I'll keep that one, you can subscribe to that issue to follow. |
Duplicates
Latest version
Steps to reproduce 🕹
Link to live example:
Minimal reproduction: https://codesandbox.io/s/recursing-cherry-9zyyjn?file=/Demo.tsx
Steps:
Current behavior 😯
The browser throws an unhandled error: Cannot read properties of null (reading 'type') with the following stack trace:
Expected behavior 🤔
Context 🔦
Our rows are "deals", i.e. open sales for our SaaS customers.
We are trying to give our users the ability to see revenue for a group (i.e. Enterprise Deals) aggregated while also filtering out irrelevant deals. However when I try to use these 3 features in combination (filtering, grouping, aggregation) I see this error.
We are a MUI premium user and would appreciate getting this issue addressed as soon as possible as it has customer impact for our small startup and is blocking future product development.
Your environment 🌎
npx @mui/envinfo
Order ID or Support key 💳 (optional)
71131
The text was updated successfully, but these errors were encountered: