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

Plans in Site Creation: Fix for the free domain name from the Calypso popup #19391

Merged
merged 5 commits into from
Oct 17, 2023

Conversation

irfano
Copy link
Member

@irfano irfano commented Oct 16, 2023

This improves the plans in the site creation flow. If a domain from the plan selection screen was received (via redirect_url), it will be used for creating the cart.

Important

Before testing and merging this PR, the Automattic/wp-calypso#83045 must be merged.

  • fa8a6b4: productName was unnecessary, so I removed it.
  • 0be3bc1: Added a domainName parameter to CreateSite event.
  • 43f7f4e: domain_name from redirect_url is passed to the CreateSite event. It's null if redirect_url does not return any domain. In that case, the domain from the domain selection will be used.

To test:

Site creation with a free domain and free plan

  1. Launch JP app
  2. Go to Me-> Debug settings
  3. Switch to the Remote Feature tab, find plans_in_site_creation, and enable it
  4. Switch to My Site tab
  5. Tap the down arrow to the right of the site name on the My Site screen
  6. Tap the + icon at the top right on the Site picker screen
  7. Select ⊕ Create WordPress.com site on the modal dialog
  8. Tap Skip at the top right on the first step (topic screen)
  9. Tap Skip at the top right on the next step (theme screen)
  10. Enter a search query (whatever you want here)
  11. Select a free domain (should be *.wordpress.com)
  12. Verify Plans screen opens with Personal, Premium, Business, and Ecommerce plans
  13. Select the free plan
  14. Verify it returns to the site setup screen
  15. Verify a site is created with the desired domain

Also, test these cases

  • Site creation with a paid domain and paid plan
  • Site creation with a paid domain and free plan
    • From popup, select the free plan
    • From popup, select the paid plan

Regression Notes

  1. Potential unintended areas of impact
    This may break different flow in site creation checkout flow

  2. What I did to test those areas of impact (or what existing automated tests I relied on)
    Tested manually

  3. What automated tests I added (or what prevented me from doing so)
    This only updates the current behavior, no automated test was added.

PR submission checklist:

  • I have completed the Regression Notes.
  • 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.

UI Changes testing checklist:

  • Portrait and landscape orientations.
  • Light and dark modes.
  • Fonts: Larger, smaller and bold text.
  • High contrast.
  • Talkback.
  • Languages with large words or with letters/accents not frequently used in English.
  • Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
  • Large and small screen sizes. (Tablet and smaller phones)
  • Multi-tasking: Split screen and Pop-up view. (Android 10 or higher)

@irfano irfano added this to the 23.6 milestone Oct 16, 2023
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 16, 2023

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr19391-3ed6555
Commit3ed6555
Direct Downloadjetpack-prototype-build-pr19391-3ed6555.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 16, 2023

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr19391-3ed6555
Commit3ed6555
Direct Downloadwordpress-prototype-build-pr19391-3ed6555.apk
Note: Google Login is not supported on these builds.

@ravishanker
Copy link
Contributor

@irfano - Tested it after merging Calypso PR. Please feel free to merge this. Give a final spin of the other PR before merging that too. 👍

@irfano irfano changed the title Use the domain name from plan selection screen if it's received Plans in Site Creation: Fix for the free domain name from the Calypso popup Oct 17, 2023
Base automatically changed from Plans-in-site-creation-Update-RELEASE-NOTES to trunk October 17, 2023 11:01
@irfano irfano requested a review from ravishanker October 17, 2023 11:12
Copy link
Contributor

@ravishanker ravishanker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@ravishanker ravishanker merged commit 2e59280 into trunk Oct 17, 2023
@ravishanker ravishanker deleted the feature/read-domain-from-plan-selection-page branch October 17, 2023 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants