-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: Integrate Plan selection into Site Creation flow #21688
Plans in Site Creation: Integrate Plan selection into Site Creation flow #21688
Conversation
- Load a special jetpack-app-plans page on Calypso - Extract selected plan id when the plan is selected
|
App Name | ![]() |
|
Configuration | Release-Alpha | |
Build Number | pr21688-c373eca | |
Version | 23.4 | |
Bundle ID | org.wordpress.alpha | |
Commit | c373eca | |
App Center Build | WPiOS - One-Offs #7361 |
|
App Name | ![]() |
|
Configuration | Release-Alpha | |
Build Number | pr21688-c373eca | |
Version | 23.4 | |
Bundle ID | com.jetpack.alpha | |
Commit | c373eca | |
App Center Build | jetpack-installable-builds #6395 |
Web view is already showing the title
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @staskus, I ran into one issue while testing and didn't review the code changes. I can re-review once this is ready again and look through the code changes.
Free Domain + Free Plan
&paid_domain_name=abc.com
query parameter makes the Free plan appear, but this doesn't seem right.
I used the Calypso live build from Automattic/wp-calypso#82404, but it seems to work if I use the live build from the original PR, Automattic/wp-calypso#82135. The only difference I see between them is a change to enabled logged out viewing of the plans page, Automattic/wp-calypso@01afb54. Could this be causing an issue?
Free Domain + Paid Plan
Works as expected: The checkout is shown with the selected plan.
Paid Domain + Free Plan
Works as expected: No checkout shown
Paid Domain + Paid Plan
Works as expected: The checkout is shown with the selected plan and domain.
I guess this change was added to allow the PR to be tested within the JP app (which doesn't know how to authenticate Calypso Live urls, so needs the page to be accessible publicly). I'm not sure why that makes it not show the Free plan, but it seems related. So the above comment isn't a real issue once your other Calypso PR is merged, it's just that it's not possible to test the Free plan + Free domain scenario. |
Yes, I can only integrate the logged-out version for testing on the app due to authentication logic but that build is not showing free plans. Sorry for the confusion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for clarifying, I see now the issue of the Free plan not showing up was already raised outside of this PR. Sorry for the confusion on my part.
The code changes look good, so I think this is ready to merge once the Calypso PR is merged and the URL used in this PR is updated.
Generated by 🚫 dangerJS |
Fixes #21661
Integrate Plan selection into Site Creation flow.
To test:
Free Domain + Free Plan
Plans in Site Creation
feature flagFree Domain + Paid Plan
Plans in Site Creation
feature flagPaid Domain + Free Plan
Plans in Site Creation
feature flagPaid Domain + Paid Plan
Plans in Site Creation
feature flagSimulator.Screen.Recording.-.iPhone.15.-.2023-10-04.at.16.24.38.mp4
Paid Domain
Regression Notes
None, it's a new functionality
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR submission checklist:
RELEASE-NOTES.txt
if necessary.UI Changes testing checklist: