Skip to content
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

Onboarding: display a clear message/CTA when website creation process fails #96827

Open
Robertght opened this issue Nov 27, 2024 · 3 comments
Open
Labels
[Feature Group] Signup & Site Onboarding Tools for user registration and onboarding new users to the site. [Feature] Site Goals & Onboarding Paths The onboarding flows for all new sites. [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Status] Escalated to Product Ambassadors [Type] Bug

Comments

@Robertght
Copy link

Quick summary

Coming from here and pfYzsZ-45-p2 I'm opening this report separately.

Steps to reproduce

N/A

@ellatrix if you remember more about this and what steps we can take to replicate, please edit this section.

What you expected to happen

To see a button to refresh or a message that the website creation process failed.

What actually happened

The screen remained stuck on loading with errors in the browser console:

Image

Impact

One

Available workarounds?

No and the platform is unusable

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@Robertght Robertght added [Feature Group] Signup & Site Onboarding Tools for user registration and onboarding new users to the site. [Feature] Site Goals & Onboarding Paths The onboarding flows for all new sites. [Product] WordPress.com All features accessible on and related to WordPress.com. [Type] Bug Needs triage Ticket needs to be triaged labels Nov 27, 2024
@Robertght Robertght removed the Needs triage Ticket needs to be triaged label Nov 27, 2024
@github-actions github-actions bot added the [Pri] High Address as soon as possible after BLOCKER issues label Nov 27, 2024
@Robertght Robertght changed the title Onboarding: display a clear message when website creation process fails Onboarding: display a clear message/CTA when website creation process fails Nov 27, 2024
@jartes jartes moved this from Needs Triage to In Triage in Automattic Prioritization: The One Board ™ Nov 27, 2024
@jartes jartes added [Pri] Low Address when resources are available. and removed [Pri] High Address as soon as possible after BLOCKER issues labels Nov 27, 2024
@jartes
Copy link
Contributor

jartes commented Nov 27, 2024

📌 REPRODUCTION RESULTS

  • Could Not Replicate

📌 FINDINGS/SCREENSHOTS/VIDEO

  • After following the steps on pfYzsZ-45-p2 I was not able to get the error.

📌 ACTIONS

  • Leaving as In Triage waiting for reproduction steps.
  • Lowering priority from High to Low.

@tanjoymor
Copy link

While I'm not sure about steps to reproduce, I can confirm that I have randomly run into this during other testing processes.

@paulopmt1 would you say this is related to what we ran into during the Quake plugin walk through?

It's also similar to what I ran into and commented on here: #96513 (comment)

Not sure what next steps should be here.

@paulopmt1
Copy link
Contributor

paulopmt1 commented Dec 16, 2024

would you say this is related to what we ran into during the Quake plugin walk through?
Could Not Replicate

By looking at the image, both https://doubleclick.net/ and public-api.wordpress.com were unreachable by the user, which indicates they had a network connection error during that process. Since these resources are from two different companies, this seem to be related to the user end.

After understanding the flow and a few attempts to reproduce the issue, I found that if the /onboarding-customation route fails for any reason, we end up in this infinite loading screen. This is the code responsible for it.

To fix it, we need to handle that Promisse.all failure event so we can either try it again, or show a retry button to the user so the window.location.assign will be called as expected.

Here's a video showing how I simulated the bug: pfYzsZ-45-p2#comment-963
(I basically renamed that /onboarding-customation request so our API returned an invalid path).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature Group] Signup & Site Onboarding Tools for user registration and onboarding new users to the site. [Feature] Site Goals & Onboarding Paths The onboarding flows for all new sites. [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Status] Escalated to Product Ambassadors [Type] Bug
Projects
Development

No branches or pull requests

4 participants