-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Update text on site credentials page #94722
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
subHeaderText={ | ||
isEnglishLocale | ||
? translate( | ||
'Please share the following details to access your site and start your migration to WordPress.com.' |
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.
The text in the image is "Drop your details below and we'll get started with your migration to WordPress.com.", but your text also looks good. Have we decided to go with this one instead?
I'm pretty sure the designs are stale, because the "Current" screenshot in the post had stale text as well - p9Jlb4-ee4-p2 |
You're right, I noticed that too. But just to make sure, I also checked Figma (F8foKv974Gc5dTHSh0i0Wo-fi-396_7998). But it's a small issue and we're making frequent updates here. So I'll just go ahead and approve. |
This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/16871433 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 @donnapep for including a screenshot in the description! This is really helpful for our translators. |
* Update text on credentials page * Update tests * Fix additional test case
Translation for this Pull Request has now been finished. |
Related to #94705.
Proposed Changes
Site address
toCurrent site address
.Skip, I need help providing access
toI need help, please contact me
.Why are these changes being made?
Testing Instructions
Pre-merge Checklist