-
-
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
[pickers] v7: Move the exports of the calendarHeader
slot to @mui/x-date-pickers/PickersCalendarHeader
#11020
[pickers] v7: Move the exports of the calendarHeader
slot to @mui/x-date-pickers/PickersCalendarHeader
#11020
Conversation
…ate-pickers/PickersCalendarHeader
Deploy preview: https://deploy-preview-11020--material-ui-x.netlify.app/ Updated pages: |
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.
LGTM ... I'll add the breaking change to the description so it can be read for CHANGELOG generation
docs/data/migration/migration-pickers-v6/migration-pickers-v6.md
Outdated
Show resolved
Hide resolved
@michelengelen I add the BC just before merging to make sure it's up to date with the migration guide, no need to do it 👍 |
Co-authored-by: Michel Engelen <32863416+michelengelen@users.noreply.github.com> Signed-off-by: Flavien DELANGLE <flaviendelangle@gmail.com>
Well ... too late! 😂 but I'll remember for the next time! |
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.
Nice, LGTM! 🎉
Breaking change
Change the imports of the
calendarHeader
slotThe imports related to the
calendarHeader
slot have been moved from@mui/x-date-pickers/DateCalendar
to@mui/x-date-pickers/PIckersCalendarHeader
:Closes #11019