Skip to content

[website] Exclude experiment pages in production#35180

Merged
siriwatknp merged 9 commits intomui:masterfrom
siriwatknp:docs/skip-experiments
Nov 21, 2022
Merged

[website] Exclude experiment pages in production#35180
siriwatknp merged 9 commits intomui:masterfrom
siriwatknp:docs/skip-experiments

Conversation

@siriwatknp
Copy link
Member

@siriwatknp siriwatknp added website Pages that are not documentation-related, marketing-focused. type: regression A bug, but worse, it used to behave as expected. 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 type: bug It doesn't behave as expected. 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
@oliviertassinari oliviertassinari removed the type: bug It doesn't behave as expected. label Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: regression A bug, but worse, it used to behave as expected. website Pages that are not documentation-related, marketing-focused.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants