i18n: Update importer/upgrade-plan-details to use numberFormat for percent values in strings#100958
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. |
| <ul>{ hostingDetailsItems }</ul> | ||
| </div> | ||
| <div className="import__upgrade-plan-hosting-details-testimonials-container"> | ||
| <p>{ translate( '100% loved by our best customers' ) }</p> |
There was a problem hiding this comment.
@agrullon95 can you help me test this? I cannot seem to be able to reach the step/location where this renders. I looked through a recent PR of yours (#89497) but no luck. 🙏
There was a problem hiding this comment.
Hi @chriskmnds! 👋
I would recommend you to check the flows to see if this step is still used, or we just forgot to remove it in a refactor. Trying to navigate quickly, I wasn't able to see the step either.
This is the pricing step I see now:
When we worked on that, we had a step similar to this one to present the pricing:
For more context, I found a PR where I worked on that old version: #90966
|
@chriskmnds - This is how it looks on your branch: Local branch (trunk): |
|
Oh thank you both @agrullon95 @renatho for confirming and testing! 💯 Really appreciate it |
|
This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/17287201 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 @chriskmnds for including a screenshot in the description! This is really helpful for our translators. |
4227a84 to
d0db7e6
Compare
|
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 |




Part of addressing https://github.com/Automattic/i18n-issues/issues/949
Proposed Changes
Updates the plan-upgrade details during site importer or plan upgrade to use placeholders with
numberFormatparsing for percentages in translation stringsMedia
setup/import-focused?siteSlug=[site not on business]- Before/after in EN:Why are these changes being made?
Part of addressing https://github.com/Automattic/i18n-issues/issues/949
Testing Instructions
/setup/import-focused?siteSlug=[site not on business]props to @agrullon95 & @renatho for helping with testing
Pre-merge Checklist