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 Improvements: Epilogue (select site / create a new site) + Quick Start prompt #17564

Merged
merged 78 commits into from
Nov 25, 2021

Conversation

momo-ozawa
Copy link
Contributor

@momo-ozawa momo-ozawa commented Nov 25, 2021

Fixes #17383

Description

This PR merges the feature branch for the Onboarding Improvements project into develop

Ref: pbArwn-35A-p2

How to test

Follow testing directions from:

Case JP iPhone WP iPhone WP iPad
No sites
1-3 sites
4+ sites

Regression Notes

  1. Potential unintended areas of impact
  • Login Epilogue flow
  • Quick Start
  • Site creation
  1. What I did to test those areas of impact (or what existing automated tests I relied on)
  • Tested the flows above manually
  1. What automated tests I added (or what prevented me from doing so)

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Nov 25, 2021

You can trigger an installable build for these changes by visiting CircleCI here.

@momo-ozawa momo-ozawa force-pushed the epic/onboarding-improvements branch from aabd9c1 to 9e32d77 Compare November 25, 2021 12:25
@momo-ozawa momo-ozawa marked this pull request as ready for review November 25, 2021 12:25
@peril-wordpress-mobile
Copy link

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@leandroalonso
Copy link
Contributor

@momo-ozawa I found one small issue that doesn't prevent this to be merged on develop (I'd say it's an edge case). Basically, when the user is logged in and adds a self-hosted site we show the epilogue to them.

Steps to reproduce

  1. Login
  2. Add a self-hosted site
  3. Notice that the epilogue is show (asking the user to select a site or to create a new one)
  4. The prompt is displayed

@momo-ozawa momo-ozawa merged commit 2a367e4 into develop Nov 25, 2021
@momo-ozawa momo-ozawa deleted the epic/onboarding-improvements branch November 25, 2021 14:05
@momo-ozawa
Copy link
Contributor Author

@leandroalonso (cc: @osullivanchris)

I found one small issue that doesn't prevent this to be merged on develop (I'd say it's an edge case). Basically, when the user is logged in and adds a self-hosted site we show the epilogue to them.

Do you think it would make the most sense to just dismiss the login vc and go back to mysite w/o showing the epilogue?
FWIW, in 18.7 we show the epilogue after adding a self-hosted site. But given the new epilogue designs, it doesn’t make much sense to show a choose site / create a new site option.

@leandroalonso
Copy link
Contributor

Do you think it would make the most sense to just dismiss the login vc and go back to mysite w/o showing the epilogue?

I guess so, yes. Unless it's super complicated!

@osullivanchris
Copy link

Good spot @leandroalonso . Proposed solution sounds good to me 👍

@AliSoftware
Copy link
Contributor

Kudos @momo-ozawa for testing and including screenshots to JPiOS (and not just WP) 👏 , which already answers the question I almost had when drafting the release notes: with the clear PR description and screenshots I know without a doubt that this does apply to both WP and JP (and should thus be included in JP release notes too) 🙇 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Onboarding Improvements: Project Issue
4 participants