Description
Fixes Automattic/martech#1649
Proposed Changes
The full spec is in Automattic/martech#1649.
Domains Step
- Change in the free domain explainer copy:
![image](https://cdn.statically.io/img/user-images.githubusercontent.com/5436027/231076707-c5d64817-fa15-4d4b-9ce1-c989c4a409fc.png)
Plans Step
- Change in the domain feature copy for paid plans:
![image](https://cdn.statically.io/img/user-images.githubusercontent.com/5436027/231077206-d25917e2-606c-4e60-882e-7e64ece45eb9.png)
- Change in the domain feature copy for the free plan. The
.wordpress.com
subdomain is fetched via a new hook useGetWordPressSubdomain
.
![image](https://cdn.statically.io/img/user-images.githubusercontent.com/5436027/231077480-90faa318-4e97-40df-a199-8e19eb221039.png)
- The free plan + paid domain modal when a paid domain has been selected, and the user clicks on the Free plan CTA. This uses the same hook, and the value is cached for 5 minutes, so in most cases, a separate network is not required to fetch the subdomain.
![image](https://cdn.statically.io/img/user-images.githubusercontent.com/5436027/231676748-7b5784df-0070-480a-903d-a19a61ce077d.png)
Testing Instructions
- Go to
/start
.
- On the domains step, search for some domains. Confirm that the free-domain explainer text (in green) matches the spec.
- Select a paid domain.
- On the plans step, confirm that the domain feature for paid plans matches the spec.
- Confirm that a
wordpress.com
subdomain is displayed under the crossed-out domain name in the feature list of the free plan.
- Click on the
Start with Free
CTA.
- Confirm that the modal is shown, and the subdomain matches the one shown on the plans grid.
- Confirm that the price shown for the Personal plan is correct.
- Select
Get Personal
, and confirm that you are taken to the checkout page with the selected paid domain and the Personal plan in the cart.
- Repeat steps 1-8.
- Select “Continue with Free”.
- Confirm that a free site is created with free subdomain shown on the modal.
Pre-merge Checklist
- Has the general commit checklist been followed? (PCYsg-hS-p2)
- Have you written new tests for your changes?
- Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
- Have you checked for TypeScript, React or other console errors?
- Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
- Have we added the “[Status] String Freeze” label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
- For changes affecting Jetpack: Have we added the “[Status] Needs Privacy Updates” label if this pull request changes what data or activity we track or use (p4TIVU-ajp-p2)?
Unfortunately, no screenshots were provided by the developer.