Conversation
Details of bundle changes.Comparing: 05c992e...67d1327
|
oliviertassinari
left a comment
There was a problem hiding this comment.
Early overview while it's still in draft. Feel free to ignore it.
9a884c7 to
fad482d
Compare
f3bf5a0 to
0a143b9
Compare
| "material-table": "^1.50.0", | ||
| "material-ui-popup-state": "^1.4.1", | ||
| "next": "8.1.0", | ||
| "next": "9.1.4", |
There was a problem hiding this comment.
Upgrade to latest stable next release in a follow-up since they changed how they polyfill.
oliviertassinari
left a comment
There was a problem hiding this comment.
The only issue I can spot on the live preview is the open side nav on the home page
Right forgot about that one. Will fix.
Not sure what you mean by production. It's already built for the production environment in the deploy previews. |
Awesome, great work!
I had http://material-ui.com/ in mind. So I can run SEO crawlers and tests the locales (e.g. /zh/) |
Why can't you do that with the deploy preview? |
2e342d3 to
67d1327
Compare
|
@eps1lon I have rebased to resolve the conflict with master, I hope it's fine with you. I think that it's good to land, feel free to merge when you are ready 🚀😄. |
|
Let's see if this works. |
Preview: https://deploy-preview-18441--material-ui.netlify.com/
/apimoves to/api-docsto not be considered an API route. Via rewrites we keep the URLs/staticis deprecated in favor of/publicWe can use this to move the
/cdninto/public. I didn't touch the static paths since this affects some 300 files some of which are translated.Containeris deprecated so I removed itFuture work
investigate if we can remove the custom server.js by using rewrites for i18nDone