[DataGrid] Change isCellEditable params type #12896
Open
+1
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi MUI-X team, thanks for your hard work.
I believe I have come across an incorrect type for
isCellEditable
'sparams
, see the diff.The bug
As the types currently are on master (and in production) ,
params.row
isany
, meaning any typo or wrong property onparams.row
won't be picked up by TS, despite users' efforts to type their grid strongly:This also means
params.value
is, I believe, incorrectly typed, making it look like the value has the row's type (which it shouldn't):The fix
With my fairly simple MR, Typescript understands what's happening and warns the user against potential bug:
That's pretty much it.