Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Untangling: apply new design to Settings -> Administration -> Transfer site #96527

Closed fushar closed 2 days ago

fushar commented 3 days ago

Fixes https://github.com/Automattic/dotcom-forge/issues/9860

Proposed Changes

This PR applies the new global view design to Settings -> Administration -> Transfer site flow.

Why are these changes being made?

pbxlJb-6ye-p2

Testing Instructions

  1. Open /sites.
  2. Click a site.
  3. Go to Settings -> Administration.
  4. Click the Transfer site button.
  5. Verify you see this:
image
  1. Enter another username of yours and click Continue. Verify you see:
image
  1. Tick all toggles and click Start transfer. Verify you see:
image
  1. Check your email with the subject Confirmation needed to finish blog transfer. Click the button:
image
  1. You will get redirected to https://wordpress.com/settings/start-site-transfer/:slug?site-transfer-confirm=code.
  2. Manually change the URL to https://<calypso-live-url>/sites/settings/administration/:slug/transfer-site?site-transfer-confirm=code. Verify you see:
image
  1. Go to the recipient's email. Accept the transfer. Verify that the is transferred successfully.
  2. REGRESSION: Transfer another site via /settings/general/:slug; verify the flow can still be completed.

Pre-merge Checklist

github-actions[bot] commented 3 days ago
Calypso Live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125054
Jetpack Cloud live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125054&env=jetpack
Automattic for Agencies live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125054&env=a8c-for-agencies
matticbot commented 3 days ago

Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:

Sections (~36 bytes added 📈 [gzipped])

``` name parsed_size gzip_size settings -150 B (-0.0%) +28 B (+0.0%) site-settings +48 B (+0.0%) +47 B (+0.0%) ``` Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to.

Legend

What is parsed and gzip size? **Parsed Size:** Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. **Gzip Size:** Compressed size of the JS and CSS files. This much data needs to be downloaded over network.

Generated by performance advisor bot at iscalypsofastyet.com.

matticbot commented 3 days ago

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

To test WordPress.com changes, run install-plugin.sh $pluginSlug untangling/transfer-site on your sandbox.

a8ci18n commented 2 days ago

This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/17009403

Some locales (Hebrew, Japanese) have been temporarily machine-translated due to translator availability. All other translations are usually ready within a few days. Untranslated and machine-translated strings will be sent for translation next Monday and are expected to be completed by the following Friday.

Thank you @fushar for including a screenshot in the description! This is really helpful for our translators.