[data grid] Aggregated columns don't respond to changes in the "editable" property of the GridColDef about mui-x HOT 7 CLOSED

@scamden I've added it for today's grooming session so we can discuss and see how to proceed.

from mui-x.

@JohannesByle the problem is not with the property itself but rather the way you control it. In order to change column configuration once the grid is initially loaded you need to use apiRef.current.updateColumns method. This is explained in the docs here https://mui.com/x/react-data-grid/column-definition/ (right below the first example).

from mui-x.

Thanks for the quick response. I had read that section of the docs, but because the two different columns behaved differently I assumed it was a bug.

In my case I have props that control whether or not a column is editable. Would the correct approach be to update the columns in a useEffect like this?

  const {editable} = props;

  useEffect(() => {
    const cols = baseCols.map((c)=> ({...c, editable}));
    apiRef.current.updateColumns(cols);
  }, [editable]);

from mui-x.

After doing further testing it looks like it's broken for the apiRef.current.updateColumns as well:
https://codesandbox.io/s/interesting-wiles-mr5vx5?file=/src/demo.tsx

from mui-x.

Ah ok, I can confirm, It doesn't seem to be working.

from mui-x.

this is a fairly huge bug.. any update on progress?

from mui-x.

This is a regression from #9334

v6.9.2: https://codesandbox.io/s/shy-wildflower-xqzpqp
v6.10.2: https://codesandbox.io/s/angry-david-ncrxph

The relevant change is here: https://github.com/mui/mui-x/pull/9334/files#diff-d6677e3149e60f2ede0c6ad76a8014630705d0668b5d8c92284a8e643d75a145

from mui-x.