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] Exclude experiment pages in production #35180

Merged
merged 9 commits into from Nov 21, 2022

Conversation

siriwatknp
Copy link
Member

@siriwatknp siriwatknp added website Pages that are not documentation-related, marketing-focused. regression A bug, but worse labels Nov 17, 2022
@mui-bot
Copy link

mui-bot commented Nov 17, 2022

Messages
📖 Netlify deploy preview: https://deploy-preview-35180--material-ui.netlify.app/

No bundle size changes

Generated by 🚫 dangerJS against d157ec5

@siriwatknp siriwatknp marked this pull request as draft November 17, 2022 06:17
@siriwatknp
Copy link
Member Author

@oliviertassinari currently, the experiment pages work in https://mui.com/experiments/, is this what you expected?

@oliviertassinari
Copy link
Member

oliviertassinari commented Nov 17, 2022

@siriwatknp Ah, awesome. I didn't intend this change in #34259, so it looks like I introduced a regression. I assume you worked on this because of #35173?

@oliviertassinari oliviertassinari marked this pull request as ready for review November 17, 2022 09:59
@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Nov 17, 2022
@siriwatknp
Copy link
Member Author

@siriwatknp Ah, awesome. I didn't intend this change in #34259, so it looks like I introduced a regression. I assume you worked on this because of #35173?

Not entirely but it reminds me to fix this problem because I tried to open the experiment pages in the deploy preview PRs but it did not work 😅.

@oliviertassinari
Copy link
Member

@siriwatknp Looks good, I have made a few light changes, trying to further improve it.

@siriwatknp siriwatknp merged commit 556d3b0 into mui:master Nov 21, 2022
alexfauquette pushed a commit to alexfauquette/material-ui that referenced this pull request Nov 22, 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
bug 🐛 Something doesn't work regression A bug, but worse website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants