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

Move adapter to mui theme #1489

Closed
wants to merge 11 commits into from
Closed

Conversation

dmtrKovalenko
Copy link
Member

A part of vNext #1293

User facing changes

No more MuiPickersUtilsProvider . Utils are coming from material-ui-theme

-<MuiPickersUtilsProvider locale={ru} utils={DateFnsUtils}>
- <ThemeProvider theme={createMuiTheme({ ... })}>
-    ...
-  </ThemeProvider>
-</MuiPickersUtilsProvider>
+ <ThemeProvider theme={createMuiTheme({  dateAdapter: new DateFnsUtils({ locale: ru }), ... })}>
+    ...
+  </ThemeProvider>

Blocked by https://github.com/mui-org/material-ui/pull/19542

@vercel
Copy link

vercel bot commented Feb 3, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/mui-org/material-ui-pickers/qrl7mjsn2
✅ Preview: https://material-ui-pickers-git-feature-move-adapter-to-mui-theme.mui-org.now.sh

@cypress
Copy link

cypress bot commented Feb 3, 2020



Test summary

37 1 0 14


Run details

Project material-ui-pickers
Status Failed
Commit a597ab0
Started Feb 3, 2020 12:54 PM
Ended Feb 3, 2020 12:55 PM
Duration 01:02 💡
OS Linux Debian - 9.8
Browser Electron 61

View run in Cypress Dashboard ➡️


Failures

e2e/integration/DatePicker.spec.ts Failed
1 DatePicker > Mobile DatePicker > Should open date picker on 01-01-2018

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@oliviertassinari oliviertassinari deleted the feature/move-adapter-to-mui-theme branch March 24, 2020 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant