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

Warm Welcome: appears twice and causes error #8723

Closed
alisterscott opened this issue Feb 2, 2018 · 5 comments · Fixed by #9010
Closed

Warm Welcome: appears twice and causes error #8723

alisterscott opened this issue Feb 2, 2018 · 5 comments · Fixed by #9010
Assignees
Labels
Admin Page React-powered dashboard under the Jetpack menu Connect Flow Connection banners, buttons, ... [Feature] Warm Welcome [Focus] FixTheFlows [Pri] High [Type] Bug When a feature is broken and / or not performing as intended

Comments

@alisterscott
Copy link
Contributor

alisterscott commented Feb 2, 2018

Steps to reproduce the issue

  1. Upgrade a site to a higher plan in Calypso
  2. Click start exploring your features
  3. wp-admin appears with overlay - close overlap
  4. Use a link to visit an external page - eg. WordPress.com stats
  5. Use back button to return to wp-admin

What I expected

Overlay to not appear again as I've already dismissed it

What happened instead

Overlay appears again. Closing it creates a red error. See sceen recording attached.

Versions

Tested 5.7.1 and 5.8 in Firefox and Chrome

Screen recording

upgrade

@alisterscott alisterscott added the [Type] Bug When a feature is broken and / or not performing as intended label Feb 2, 2018
@alisterscott
Copy link
Contributor Author

This bug also appears during the screen recording: Automattic/wp-calypso#22018

@alisterscott
Copy link
Contributor Author

This bug also appears during the screen recording: Automattic/wp-calypso#22034 (I pressed refresh)

@alisterscott alisterscott changed the title Upgrading overlaps appears twice and causes error Upgrading overlays appears twice and causes error Feb 2, 2018
@alisterscott
Copy link
Contributor Author

@jeherve I'm not sure what labels to add for this, I couldn't find a plans one?

@jeherve jeherve changed the title Upgrading overlays appears twice and causes error Warm Welcome: appears twice and causes error Feb 2, 2018
@jeherve jeherve added Admin Page React-powered dashboard under the Jetpack menu [Pri] High [Feature] Warm Welcome labels Feb 2, 2018
@jeherve
Copy link
Member

jeherve commented Feb 2, 2018

Related: #7837

@alisterscott There is no Plans label yet (I jus added one so we can use it in the future), but "Warm Welcome" (the name of the feature) should do the trick. I added it.

@designsimply
Copy link

Tested and confirmed using the steps provided.

I also noticed the following request and response related to the 400 error that shows up when you dismiss the overlay the second time:

Request URL: https://convenient-cuban.jurassic.ninja/wp-json/jetpack/v4/settings
Response payload: {"code":"some_updated","message":"","data":{"status":400}}

screen shot 2018-03-08 at thu mar 8 3 25 27 pm
Seen at Developer Tools > Network panel for https://convenient-cuban.jurassic.ninja/wp-admin/admin.php?page=jetpack#/dashboard using Firefox 58.0.2 on macOS 10.13.3.

The error for reference:

Error updating settings. (Status 400)

screen shot 2018-03-08 at thu mar 8 3 35 51 pm
Seen at https://traditional-titi.jurassic.ninja/wp-admin/admin.php?page=jetpack#/dashboard running WordPress 4.9.4 and Jetpack 5.9 using Firefox 58.0.2 on macOS 10.13.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Page React-powered dashboard under the Jetpack menu Connect Flow Connection banners, buttons, ... [Feature] Warm Welcome [Focus] FixTheFlows [Pri] High [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants