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

[docs] Fix codesandbox export with dayjs #34619

Merged

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Oct 5, 2022

Since mui/mui-x#5481 the bug is a lot more visible. When you export a codesandbox from https://mui.com/x/react-date-pickers/date-picker/#basic-usage

Screenshot 2022-10-05 at 11 19 14

you will get https://codesandbox.io/s/3sem4m?file=/package.json:161-172

Screenshot 2022-10-05 at 11 19 44

but this package doesn't exist. Dayjs already includes the types, proof: https://unpkg.com/browse/dayjs@1.11.5/index.d.ts.


I found this while I was doing a bit of support in https://groups.google.com/a/mui.com/g/x/c/xHe-_HUVhjc.

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Oct 5, 2022
@mui-bot
Copy link

mui-bot commented Oct 5, 2022

No bundle size changes

Generated by 🚫 dangerJS against dbe0b00

@oliviertassinari oliviertassinari merged commit f235e8e into mui:master Oct 5, 2022
@oliviertassinari oliviertassinari deleted the fix-dayjs-codesandbox branch October 5, 2022 09:55
@flaviendelangle
Copy link
Member

Will be applied to MUI-X in mui/mui-x#6103

@oliviertassinari oliviertassinari added the scope: docs-infra Specific to the docs-infra product label Oct 5, 2022
alexfauquette pushed a commit to alexfauquette/material-ui that referenced this pull request Oct 14, 2022
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
feliperli pushed a commit to jesrodri/material-ui that referenced this pull request Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation MUI X scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants