Skip to content
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

[PickersCalendarSkeleton] Rename to CalendarPickerSkeleton #25679

Merged
merged 5 commits into from
Apr 12, 2021

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Apr 9, 2021

BREAKING CHANGE

- import PickersCalendarSkeleton from '@material-ui/lab/PickersCalendarSkeleton';
+import CalendarPickerSkeleton from '@material-ui/lab/CalendarPickerSkeleton';

Based on #25680 (diff)

In preparation of https://github.com/mui-org/material-ui/issues/25481 where want to rename DayPicker to CalendarPicker.

The new name makes the relation to CalendarPicker more obvious.

Discovered that we don't cleanup api-docs so I'll do that separately.

@eps1lon eps1lon added breaking change component: pickers This is the name of the generic UI component, not the React module! labels Apr 9, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Apr 9, 2021

Details of bundle changes

Generated by 🚫 dangerJS against 2f5d7ac

@eps1lon eps1lon force-pushed the feat/rename-calendars-picker-skeleton branch from 48de94d to 6a7e422 Compare April 9, 2021 10:25
@eps1lon eps1lon marked this pull request as ready for review April 9, 2021 10:29
@eps1lon eps1lon force-pushed the feat/rename-calendars-picker-skeleton branch from 6a7e422 to 2f5d7ac Compare April 9, 2021 13:23
@@ -1252,6 +1257,38 @@ function generateApiPagesManifest(outputPath: string, prettierConfigPath: string
writePrettifiedFile(outputPath, source, prettierConfigPath);
}

async function removeOutdatedApiDocsTranslations(components: ReactApi[]): Promise<void> {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we will merge #25680 first

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on #25680 (diff)

@eps1lon eps1lon added the on hold There is a blocker, we need to wait label Apr 12, 2021
@eps1lon
Copy link
Member Author

eps1lon commented Apr 12, 2021

Will merge after todays release. This change only makes sense with #25685

@eps1lon eps1lon merged commit 9e097a5 into mui:next Apr 12, 2021
@eps1lon eps1lon deleted the feat/rename-calendars-picker-skeleton branch April 12, 2021 12:15
@eps1lon eps1lon removed the on hold There is a blocker, we need to wait label Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: pickers This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants