-
-
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
[survey] Remove survey promotion items #30122
Conversation
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.
👍
Should we remove the notification too? |
I'd suggest reverting the previous commits rather than manually removing the changes. |
What's the difference between this approach and reverting the commits? I really don't know. |
One point I wonder about: Should we leave some infrastructure in place for the next time we want to add a banner? For instance, we have kept
I think that it's well covered in https://www.atlassian.com/git/tutorials/undoing-changes/git-revert. |
I just meant for the future. It's quicker and easier, and less likely to miss something. Also easier to review, as you're only concerned with checking that the appropriate commits are reverted. |
Or at any point. The docs shouldn't published at a newer commit in the meantime, as that risks the content being out of sync with the published packages. |
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.
If we want to keep the banner generic, we should still import it from all the pages we had before, just use the new name WebsiteBanner
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.
Updated to use feature toggle to hide the banner.
Nice new |
Note: this PR is not enough to remove all the survey, it's still present in MUI X, e.g. https://mui.com/components/data-grid/editing/ It's a bit confusing to see the banner, clicking on it to have: cc @mui-org/x |
The survey card isn't the only confusing thing though, https://mui.com/components/data-grid/ is still showing to me as v5.2.3, click in any Core component and then v5.2.5. So maybe it's something even bigger than the scope of this PR?! |
Yes, definitely. It suggests that we need to upgrade the dependency on the mono-repo in X, it hasn't been updated for > 3 weeks. I wonder if renovate could do it for us every week. |
To be merged before this week's release as the survey closes for answers on Tuesday, Dec 14th.