Settings: open the remaining support links as WordPress.com docs in the Help Center on Dotcom sites - #52560
Conversation
…he Help Center on Dotcom sites Follow-up to #52436, which covered the info popovers. The inline support links in the settings cards still pointed at jetpack.com (or akismet.com) on WordPress.com sites. Add a small `components/support-link` helper (`SupportLink`, `getSupportUrl`, `openWpcomSupportDoc`) with the same rule as `SupportInfo`: WordPress.com sites get the WordPress.com doc in the Help Center, self-hosted sites keep the Jetpack doc in a new tab. - Anti-spam: popover linked to akismet.com; add the WordPress.com doc. - Related Posts: inline "block editor" link and the block-theme card. - Stats: "Learn More" shown while Stats is inactive. - Site Verification: "full instructions" link. - Google Analytics: "find your Measurement ID" link. The old redirect anchor (#step-2-…) does not exist on the doc; use #get-your-measurement-id. - Earn: the PayPal CTA card. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! Jetpack plugin: The Jetpack plugin has different release cadences depending on the platform:
If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. |
Code Coverage SummaryCoverage changed in 4 files.
1 file is newly checked for coverage.
|
Cover the new support-doc branches in `RelatedPostsComponent` and `EarnFeatureButton` (exported for testing) for the coverage check. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Follow-up to #52436, which made the "Learn more" links in the Settings info popovers open WordPress.com support docs in the Help Center on WordPress.com sites. Several inline support links in the settings cards still pointed at jetpack.com (or akismet.com) on those sites.
Proposed changes
_inc/client/components/support-linkhelper with the same rule asSupportInfo: on WordPress.com (Simple/Atomic) sites,SupportLinkrendersWpcomSupportLinkso the doc opens inside the Help Center; everywhere else it opens the Jetpack doc in a new tab.getSupportUrl()/openWpcomSupportDoc()do the same for non-link elements such asCardCTAs. Unit tests included.wordpress.com/support/protect-from-spam/on WordPress.com sites.wordpress.com/support/related-posts/.wordpress.com/support/stats/.wordpress.com/support/site-verification-services/.wordpress.com/support/google-analytics/#get-your-measurement-id. The existing redirect's anchor (#step-2-get-your-measurement-id) does not exist on that doc;#get-your-measurement-iddoes, and the Help Center scrolls to it.wordpress.com/support/wordpress-editor/blocks/pay-with-paypal/.Left unchanged on purpose: Backup, Scan, Search, Carousel, Jetpack Blocks and Blaze popovers (no WordPress.com doc specified), the SSO "two-step enforced" note, the Google Analytics "Anonymize IP" link (points at Google's own help), and "Enable targeted advertising".
Related product discussion/links
Does this pull request change what data or activity we track or use?
No. Existing
recordJetpackClicktracking on the converted Card CTAs is preserved.Testing instructions
WordPress.com (Atomic)
jetpack rsync).Self-hosted Jetpack
Verified on an Atomic site for all links except the two that need Stats off / a block theme, which are covered by the unit tests.
🤖 Generated with Claude Code