-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[core] Batch small fixes #28381
[core] Batch small fixes #28381
Conversation
579986a
to
c7a2a58
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have just one question, the other changes look good!
ce135b8
to
015b442
Compare
@@ -10,7 +10,7 @@ | |||
https://next.material-ui.com/* https://mui.com/:splat 301! | |||
|
|||
# Support multiple domains | |||
https://material-ui.dev/* https://mui.com/:splat 301! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also wouldn't have worked without material-ui.dev
being setup as an alias domain on netlify:
For Netlify to redirect from a domain that is not the site’s main custom domain, the domain must be assigned to the site — probably as a domain alias or a branch subdomain.
-- https://docs.netlify.com/routing/redirects/redirect-options/#domain-level-redirects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It used to be, I removed it about two months ago. It's a cleanup.
I have pushed one commit, with a description sometimes to get a more granular level of "why"