-
-
Notifications
You must be signed in to change notification settings - Fork 32.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
[l10n] Port locale to TypeScript #20685
Conversation
1ce7689
to
e0a046b
Compare
Details of bundle changes.Comparing: d48def3...348a01b Details of page changes
|
e0a046b
to
07ba926
Compare
A side note regarding TypeScript migration, I have identified this change to run
I don't know if it's still relevant. |
I don't see any |
Definitely, it's a broader comment. I wanted to share the information if in case somebody faces this problem in the future, hopefully, it can save us time :). |
It's unclear what this |
efe86ba
to
348a01b
Compare
@oliviertassinari Thanks for cleaning up! |
@eps1lon Does it make it our first ever published module written in TypeScript 😱? |
I guess so. |
Ok, I was starting to be anxious about the next release, but codesandbox-ci is here, it looks 👌 https://codesandbox.io/s/modest-proskuriakova-mkipj?file=/src/App.js. |
Changeset for changelog:
@material-ui/core/locale
expandText
type toBreadcrumbsProps
backIconButtonText
andnextIconButtonText
type toTablePaginationProps
labelRowsPerPage
inTablePaginationProps
tostring
(wasReactNode
). Since this is forwarded toaria-label
an actualReactNode
would not have the intended effect./core/locale is as a good as a candidate for testing "TypeScript as source language" as we can get:
Part of #15984
In general I feel pretty comfortable with migrating any non-react, non-generic implementations to typescript. But this is definitely something we should be skeptical of. I don't want to introduce code that's not being worked on because types.
/cc @mui-org/core