-
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
Editor: Welcome Tour appears even if dismissed. #57660
Comments
Yet another occurrence for TeamCity ID 6949795, with the same user. |
I've logged in as
Besides, the “Welcome Tour” will show according to your local storage as the value of |
Typically, this issue is not reproducible by simply logging into the account using a browser. In other words, this issue is intermittent, but when it occurs it is quite readily reproducible in both CI and 'normal' desktop environments. The Welcome Tour is shown despite the However, the test statistics page for the spec Note that the step cited above is just one of the multiple in the entire e2e suite that attempts to
Perhaps this is the problem, but at the same time I recall tinkering with the value there to observe the behavior, and it is possible that the values I've copied into the description were from post-tinkering. In any case, this issue is intermittently reproducible in TeamCity and the values for LocalStorage are not set for e2e runs. |
Maybe we should always run |
The failures are always at the step of obtaining the iframed editor itself, of which the Welcome Tour is a component. If the iframed editor's handle cannot be obtained as expected, how will the tour be dismissed? Edit: in my experiments I've tried using keyboard shortcuts to see if that works, to no luck. If you're aware of a shortcut that should dismiss this, please let me know. |
Oh I'm sorry for my misunderstanding. But when I dive into test statistics page and #7913, the reason of failure is the timeout of obtaining the iframe editor or its title as you said. I don't understand why the Welcome Tour blocks the retrieval of the iframed editor.
I just found we can click |
Thanks, I will see if I can try doing that to work around the issue. |
It looks like in order to dismiss this using Since the issue is with the Welcome Tour likely blocking the locating and retrieval of the iframed editor in the first place, this isn't a feasible option. I'll have to think of anther potential workaround. |
This issue has also been observed on I18N user as well, most often immediately changing the UI language. |
Reproduced again on LocalStorage:
|
The Welcome Tour seems to be disobeying the results of In Playwright, I intercept and modify the request to
However, this still results in the Welcome Tour showing. The LocalStorage values show that Welcome Tour is set to |
Running some suites in parallel locally, I've noticed that the Welcome Tour pops up every time the same site is being tested against concurrently. When running suites synchronously 1 by 1, the popup never came up. I haven't dug any deeper, but maybe it rings a bell @worldomonation @arthur791004? |
@WunderBart I've only ever run my end-to-end tests synchronously (one spec file, in order) but without the While we're on this topic, I had to come up with an alternative way to dismiss this pesky modal because the I have a WIP PR that switches from |
Frankly, this would be much easier if the |
Just noting that even in this case, the site still might be "in use" by the CI from the same account, which means that the concurrent requests are still possible. |
Thanks for your work on this folks! We may have another case here: |
Another report in https://wordpress.com/forums/topic/new-welcome-popup/ |
Another case here related to this error:
|
|
@alshakero 👋 We're seeing a recent uptick of reports on this issue. Is there anything we can do to help get this escalated for more eyes-on and a fix? Thanks! |
|
Reported in 35979867 - hc |
@cometgrrl 👋 Would it be possible for you to help nudge this along? |
@Automattic/vertex Can you take a look at this? Anecdotally, I see the welcome tour more often than I should. I started taking screenshots to keep track, and I saw it on June 20th, and again on July 1st. Both on my team P2, same browser/computer/etc. |
I'm investigating this issue. Scenario 1
Scenario 2
From what I see this is a front-end issue. |
Since the follow-ups were not a lot - I've reached the affected users manually. 👍 |
Replied to more here: 5491132-zen |
Quick summary
The welcome tour modal appears when launching the editor even if the state is marked as 'dismissed'.
Steps to reproduce
gutenbergSimpleSiteUser
. Login information is found in the encrypted configuration file.What you expected to happen
A new, blank editor is loaded.
What actually happened
Welcome Tour appears in the bottom left.
wp-editor__post-basic-flow-spec-start-new-post.mp4
Context
Automated end to end tests.
Browser
No response
Simple/Atomic
Simple
Other notes
LocalStorage (from Microsoft Edge):
Slack thread: p1636055367011200/1635542471.006000-slack-C029WGXE5T4
p1636045337071000/1635462959.039100-slack-CBTN58FTJ
p1636042344032100-slack-C01QG4Y91RR
p1635535550045200/1635462959.039100-slack-CBTN58FTJ
p1635462959039100-slack-CBTN58FTJ
Reproducibility
Intermittent
Severity
Some (< 50%)
Available workarounds?
Yes, easy to implement
Workaround details
For humans, the workaround is easy to implement.
For automated testing framework, this is a blocker as this blocks the retrieval of the iframed editor.
The text was updated successfully, but these errors were encountered: