-
Notifications
You must be signed in to change notification settings - Fork 272
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
Add onboarding modal to playground #1777
Add onboarding modal to playground #1777
Conversation
Ahh, seeing that the Cypress was failing tests, I updated the website-ui tests and added a URL param check. Looks like things are good now. 🙂 |
👋 Thank you for contributing @n8finch! It pains me to say this – the onboarding modal idea will soon be obsoleted by this work: #1731. It will enable saving Playgrounds, solve the data loss problem, and introduce a sidebar where any required onboarding could happen. We should have closed the original issue earlier, sorry about that! |
Resolves #497
Motivation for the change, related issues
Adds an onboarding modal using the
Modal
from WordPress Components.Testing Instructions (or ideally a Blueprint)
npm run dev
hideExperimentalNotice
cookie from your Application cacheFeedback
@adamziel I worked on this with @sejas , but wanted to check with you specifically on two pieces: