-
Notifications
You must be signed in to change notification settings - Fork 27
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
Remove rebranding modal pop-up to announce visual changes #3516
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: f0fc659 The changes in this PR will be included in the next version bump. This PR includes changesets to release 36 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Awesome. To be frank: I can't wait for this 😄. Let's not forget we can remove some local storage value setting from Cypress and Checkly as a follow-up of integrating this update. Just DM me if you need more information.
Thank you @tdeekens, for the reminder! I removed the local storage values from this repo here. And I'll create a follow-up ticket so we don't miss on removing it from other repos as well. |
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.
💯
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.
🙌🏾
* chore: removing rebranding modal * chore: remove local storage value for rebranding modal
We no longer need the modal that announced the rebranding changes in the MC.
PR that introduced the modal - here.