Update learn more link in Sign in with Google toggle.

[benbowler]

Summary

Addresses issue:

• Incorrect redirect when clicking on 'Learn More' link in Connected Services #9734

Relevant technical choices

PR Author Checklist

• My code is tested and passes existing unit tests.
• My code has an appropriate set of unit tests which all pass.
• My code is backward-compatible with WordPress 5.2 and PHP 7.4.
• My code follows the WordPress coding standards.
• My code has proper inline documentation.
• I have added a QA Brief on the issue linked above.
• I have signed the Contributor License Agreement (see https://cla.developers.google.com/).

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Code Reviewer Checklist

• Run the code.
• Ensure the acceptance criteria are satisfied.
• Reassess the implementation with the IB.
• Ensure no unrelated changes are included.
• Ensure CI checks pass.
• Check Storybook where applicable.
• Ensure there is a QA Brief.
• Ensure there are no unexpected significant changes to file sizes.

Merge Reviewer Checklist

• Ensure the PR has the correct target branch.
• Double-check that the PR is okay to be merged.
• Ensure the corresponding issue has a ZenHub release assigned.
• Add a changelog message to the issue.

[benbowler]

Note, as of creating this PR the URL links to 404 but I assume the documentation is yet to be published.

[github-actions]

Build files for 9bb67f2 are ready:

• Download development build
• Download production build

[github-actions]

Size Change: +20 B (0%)

Total Size: 1.94 MB

Filename	Size	Change
./dist/assets/js/googlesitekit-datastore-site-********************.js	21 kB	+28 B (+0.13%)
./dist/assets/js/googlesitekit-modules-sign-in-with-google-********************.js	26.3 kB	-8 B (-0.03%)

ℹ️ View Unchanged

Filename	Size
./dist/assets/css/googlesitekit-admin-css-********************.min.css	61 kB
./dist/assets/css/googlesitekit-adminbar-css-********************.min.css	11.8 kB
./dist/assets/css/googlesitekit-authorize-application-css-********************.min.css	846 B
./dist/assets/css/googlesitekit-wp-dashboard-css-********************.min.css	8.2 kB
./dist/assets/js/33-********************.js	2.76 kB
./dist/assets/js/34-********************.js	2.25 kB
./dist/assets/js/35-********************.js	3.64 kB
./dist/assets/js/36-********************.js	936 B
./dist/assets/js/37-********************.js	893 B
./dist/assets/js/38-********************.js	1.61 kB
./dist/assets/js/39-********************.js	1.57 kB
./dist/assets/js/40-********************.js	1.61 kB
./dist/assets/js/41-********************.js	1.59 kB
./dist/assets/js/42-********************.js	1.83 kB
./dist/assets/js/43-********************.js	3.12 kB
./dist/assets/js/analytics-advanced-tracking-********************.js	901 B
./dist/assets/js/googlesitekit-activation-********************.js	24 kB
./dist/assets/js/googlesitekit-ad-blocking-recovery-********************.js	54.9 kB
./dist/assets/js/googlesitekit-adminbar-********************.js	34.6 kB
./dist/assets/js/googlesitekit-api-********************.js	10.1 kB
./dist/assets/js/googlesitekit-components-gm2-********************.js	6.16 kB
./dist/assets/js/googlesitekit-components-gm3-********************.js	10.1 kB
./dist/assets/js/googlesitekit-consent-mode-********************.js	25.6 kB
./dist/assets/js/googlesitekit-data-********************.js	2.37 kB
./dist/assets/js/googlesitekit-datastore-forms-********************.js	8.96 kB
./dist/assets/js/googlesitekit-datastore-location-********************.js	2.08 kB
./dist/assets/js/googlesitekit-datastore-ui-********************.js	10 kB
./dist/assets/js/googlesitekit-datastore-user-********************.js	27.6 kB
./dist/assets/js/googlesitekit-entity-dashboard-********************.js	81.9 kB
./dist/assets/js/googlesitekit-events-provider-contact-form-7-********************.js	646 B
./dist/assets/js/googlesitekit-events-provider-easy-digital-downloads-********************.js	624 B
./dist/assets/js/googlesitekit-events-provider-mailchimp-********************.js	630 B
./dist/assets/js/googlesitekit-events-provider-ninja-forms-********************.js	712 B
./dist/assets/js/googlesitekit-events-provider-optin-monster-********************.js	675 B
./dist/assets/js/googlesitekit-events-provider-popup-maker-********************.js	634 B
./dist/assets/js/googlesitekit-events-provider-woocommerce-********************.js	657 B
./dist/assets/js/googlesitekit-events-provider-wpforms-********************.js	633 B
./dist/assets/js/googlesitekit-i18n-********************.js	3.93 kB
./dist/assets/js/googlesitekit-main-dashboard-********************.js	165 kB
./dist/assets/js/googlesitekit-metric-selection-********************.js	51.5 kB
./dist/assets/js/googlesitekit-modules-ads-********************.js	33.6 kB
./dist/assets/js/googlesitekit-modules-adsense-********************.js	118 kB
./dist/assets/js/googlesitekit-modules-analytics-4-********************.js	186 kB
./dist/assets/js/googlesitekit-modules-********************.js	22.2 kB
./dist/assets/js/googlesitekit-modules-pagespeed-insights-********************.js	22.6 kB
./dist/assets/js/googlesitekit-modules-reader-revenue-manager-********************.js	41.1 kB
./dist/assets/js/googlesitekit-modules-search-console-********************.js	66.2 kB
./dist/assets/js/googlesitekit-modules-tagmanager-********************.js	32 kB
./dist/assets/js/googlesitekit-notifications-********************.js	22.8 kB
./dist/assets/js/googlesitekit-polyfills-********************.js	378 B
./dist/assets/js/googlesitekit-settings-********************.js	127 kB
./dist/assets/js/googlesitekit-splash-********************.js	69.1 kB
./dist/assets/js/googlesitekit-user-input-********************.js	44.1 kB
./dist/assets/js/googlesitekit-vendor-********************.js	322 kB
./dist/assets/js/googlesitekit-widgets-********************.js	98.1 kB
./dist/assets/js/googlesitekit-wp-dashboard-********************.js	62.6 kB
./dist/assets/js/runtime-********************.js	1.4 kB

_{compressed-size-action}

[10upsimon]

LGTM ✅

[aaemnnosttv]

Thanks @benbowler – this will need updating as this issue was somewhat incompletely defined. I've gone ahead and updated the AC + IB to be more clear for consistency regarding our approach to these kinds of links.

[aaemnnosttv]

Sending back for a quick follow-up.

[benbowler]

Hey @aaemnnosttv, I had to update the getDocumentationLinkURL selector to support hash properties as well to meet the AC. Updated and back to CR.