[data grid] Row grouping does not work with nested row values about mui-x HOT 10 CLOSED

@flaviendelangle Thanks for the help and suggestions.

from mui-x.

Hey @cbrainee and thanks for raising an issue here.
Could you please provide us with a minimal working reproduction example?
It's way easier and less error prone if you could do that.

from mui-x.

https://codesandbox.io/p/sandbox/eloquent-voice-lq5rz4?file=%2Fsrc%2FDemo.js%3A18%2C13

from mui-x.

@cbrainee could you make that sandbox public? I cannot access it

from mui-x.

@michelengelen Done

from mui-x.

OK, I see that it is not working the way one would expect.
I am not familiar with how the inner workings of the row grouping look like.
@flaviendelangle you have introduced that feature if I am not mistaken
Could you please have a look here?

from mui-x.

Hi,

You need to use the groupingValueGetter prop (doc) to retrieve the grouping value.

And if some value have no org, they will remain at the top level (doc).
If you want to group them as well, you must put a default value.

Here is what the final column could look like:

        {
            field: "customerOrg",
            headerName: "Customer Org",
            width: 150,
            renderCell: (data) => {
                if (data.rowNode.type === "group") {
                    return data.value;
                }
                return <p>{data.formattedValue}</p>;
            },
            valueGetter: (value, row) => {
                return row.customerInfo?.customerOrg;
            },
            groupingValueGetter: (value, row) => {
                return row.customerInfo?.customerOrg ?? 'No organization';
            },
        },

By the way, in the renderCell, I would advise you to use data.formattedValue instead of re-accessing it from the row.

from mui-x.

By the way, you might not do this in your real application, but getRowId should be memoized, otherwise you will end up with terrible performances.
The doc is here

from mui-x.

@cbrainee if this solution fits your use-case please feel free to close the issue.

from mui-x.

The issue has been inactive for 7 days and has been automatically closed.

from mui-x.