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

[website] Rebranding go live #28039

Closed
wants to merge 1 commit into from
Closed

Conversation

siriwatknp
Copy link
Member

@siriwatknp siriwatknp commented Aug 30, 2021

This PR should be merged once we are ready to go live.

Move homepage, pricing and about to /pages/ (replace old index), anything I missed?

Preview https://deploy-preview-28039--material-ui.netlify.app/

@siriwatknp siriwatknp added the website Pages that are not documentation-related, marketing-focused. label Aug 30, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Aug 30, 2021

No bundle size changes (experimental)

Generated by 🚫 dangerJS against 2aae7f9

@oliviertassinari

This comment has been minimized.

@siriwatknp
Copy link
Member Author

@oliviertassinari I will move your comments to another refinement PR because this PR is only about switching files.

@siriwatknp siriwatknp added the on hold There is a blocker, we need to wait label Aug 31, 2021
@oliviertassinari
Copy link
Member

oliviertassinari commented Aug 31, 2021

@siriwatknp Sounds fair, I refined a bit more in #28069. It was more feedback motivated by the urgency: "oh it's going live, it's now or never" 😁.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 31, 2021
@siriwatknp
Copy link
Member Author

@oliviertassinari @mbrookes I have moved the suggestion to #28081, please continue on that PR instead. I might close this one if there are too much conflict after #27666 is merged.

@eps1lon eps1lon changed the base branch from next to master September 14, 2021 09:14
@siriwatknp
Copy link
Member Author

the switch will be in #28081

@siriwatknp siriwatknp closed this Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold There is a blocker, we need to wait PR: out-of-date The pull request has merge conflicts and can't be merged website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants