site-kit-wp icon indicating copy to clipboard operation
site-kit-wp copied to clipboard

TwG terminology changes

Open FlicHollis opened this issue 3 years ago • 4 comments

There have been some terminology changes from the TwG team as commented in this ticket https://github.com/google/site-kit-wp/issues/5755. We are moving these out to be a ticket of their own for clarity.

The TwG team have let us know that they have changed the terminology from "Publisher ID" to "Thank with Google ID."

The TwG team also don't want to use the term "Publisher Center" any longer; it should be "Thank with Google profile." So to summarize:

References to "Publisher ID" or "Publication ID" should be changed to "Thank with Google ID" References to "Publisher center" should be changed to "Thank with Google profile"


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

Acceptance criteria

  • All instances of "Publisher ID" in Thank with Google UI should be changed to "Thank with Google ID". (Note: This doesn't actually apply to Site Kit as we don't use that term for TwG anywhere. But listed here for posterity.)
  • All instances of "Publisher center" in Thank with Google UI should be changed to "Thank with Google profile".
  • Other instances of these terms outside the TwG UI should not be changed.

Implementation Brief

In assets/js/modules/thank-with-google/components/setup/SetupPublicationPendingVerification.js, assets/js/modules/thank-with-google/components/settings/SettingsView.js,

  • Search for and replace instances of "Publication ID" and "Publisher center" to the new wording, as per the AC

Test Coverage

  • N/A

QA Brief

  • In the settings view Publication ID should be changed to Thank with Google ID
  • In Setup, Pending verification Publisher Center should be changed to Thank with Google profile Eg -> Screenshot 2022-10-08 at 23 34 00

Changelog entry

  • Update Thank with Google copy in settings and setup screens.

FlicHollis avatar Oct 04 '22 13:10 FlicHollis

@tofumatt @FlicHollis Didn't manage to find any references to "Publisher ID", other than in AdSense, hence only referencing "Publication ID" in the IB. Should the AC be updated to reflect that?

maciejost avatar Oct 04 '22 21:10 maciejost

@makiost Huh, fair point.

I'll leave the ACs as-is but add a note that this was an instruction from the TwG team and that it doesn't apply to us. Maybe there are other things using that term 😅

IB ✅

tofumatt avatar Oct 06 '22 19:10 tofumatt

QA Update ⚠️

@tofumatt @sashadoes Under Thank with Google JS file I found 60 instances of 'Publication ID' . Just want to confirm - Did we change Publication ID to Thank with Google ID only for UI and internally we are still using PublicationID ?

image

  • Verified on dev.
  • Settings view Publication ID changed to Thank with Google ID.
  • Publisher Center changed to Thank with Google profile.
  • Also, verified aria lable under Set up>pending verification. It is successfully change.

image

image

mohitwp avatar Oct 13 '22 13:10 mohitwp

@mohitwp Those aren't publicly-visible strings; they're developer messages/errors that shouldn't appear in production code.

Internally we still use publicationID as a setting/option variable name; this issue is just for user-facing strings, so that's okay. 👍🏻

I'll move this to approval since the other QA is okay 🙂

tofumatt avatar Oct 13 '22 14:10 tofumatt