-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Welcome Tour showing behind "Your first post is published" modal #57702
Comments
Hmmm...I'm not sure it's a bug or not. In this flow, the Welcome Tour shows when you land on the editor. Should we need to hide the modal after "first post is published" modal appears and show it again after "first post is published" modal is close? Or you don't see the "Welcome Tour" at the beginning and it shows suddenly after you publish the post? (But I cannot reproduce this cases) |
If you can't reproduce the issue, we can just wait and see if anyone reports this again. |
Okay. Let's wait and see if anyone encounters this again! |
it looks like https://github.com/Automattic/wp-calypso/pull/57982/files is a potential fix, right @arthur791004. I couldn't reproduce it, but the theory that the iframe url was changing right after publishing sounds plausible, I think we should deploy it and then see if this keeps coming up |
Yep, it might be the potential fix. But I'm not sure it's the same situation or not. |
I'll go ahead and close this. We know to keep an eye out if it happens in the future. |
Description
In new Blogger/Write flow: Welcome Tour appeared
Steps to reproduce
What I expected to happen
To see only the "Your first post is published" modal and NOT the Welcome Tour
What happened
I saw both the "Your first post is published" modal AND the Welcome Tour
Related
A bug reported about the Welcome Tour appearing even after dismissing it: #57660. I don't think the cause of the issue reported above and #57660 are the same, but might make sense to tackle them together.
The text was updated successfully, but these errors were encountered: