-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[docs] Improve codemod for v7 #11650
Conversation
If you want to run the codemods one by one, check out the codemods included in the [preset-safe codemod for data grid](https://github.com/mui/mui-x/blob/master/packages/x-codemod/README.md#preset-safe-for-data-grid-v700) for more details. | ||
If you want to run the codemods one by one, check out the codemods included in the [preset-safe codemod for data grid](https://github.com/mui/mui-x/blob/HEAD/packages/x-codemod/README.md#preset-safe-for-data-grid-v700) for more details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong link. I think we want to always point to the latest active codebase, so HEAD.
npx @mui/x-codemod v7.0.0/pickers/preset-safe <path> | ||
npx @mui/x-codemod@next v7.0.0/pickers/preset-safe <path> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as #11521 (comment)
- <DateCalendar defaultCalendarMonth={dayjs('2022-04-01')}; | ||
+ <DateCalendar referenceDate{dayjs('2022-04-01')} /> | ||
-<DateCalendar defaultCalendarMonth={dayjs('2022-04-01')}; | ||
+<DateCalendar referenceDate{dayjs('2022-04-01')} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The format with the space looks weird. Same reason why we write:
console.log(foo);
and not:
console.log(foo);
+ cellSelection | ||
- cellSelectionModel={{ 0: { id: true, currencyPair: true, price1M: false } }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing unstable_
I opened a PR rather than pushing straight to HEAD to make sure this passes the CI. These changes are straightforward, merging. |
No description provided.