Skip to content
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

Fix the typo in the "Display visitor groups" Settings switch label. #8496

Closed
4 tasks done
techanvil opened this issue Apr 8, 2024 · 2 comments
Closed
4 tasks done
Labels
Module: Analytics Google Analytics module related issues P1 Medium priority Squad 2 (Team M) Issues for Squad 2 Type: Enhancement Improvement of an existing feature

Comments

@techanvil
Copy link
Collaborator

techanvil commented Apr 8, 2024

Feature Description

As mentioned on #8424 (comment), there was initially a typo in the copy for the "Display visitor groups" Settings switch label (it read "Display visitors groups").

The copy should be updated, in line with the updated Figma design, to read "Display visitor groups in dashboard".

While updating this copy we should also address a minor code organisation issue introduced by the same issue.


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

Acceptance criteria

  • The copy for the "Display visitor groups" Settings switch label should read: Display visitor groups in dashboard
  • Non-functional, but the SettingsCardVisitorGroups component should be moved to either assets/js/modules/analytics-4/components/settings/AudienceSegmentation/, in line with the initial code organization section in the design doc, or, to assets/js/modules/analytics-4/components/audience-segmentation/settings/ if 8495 has been implemented before this issue.

Implementation Brief

  • In assets/js/components/settings/SettingsCardVisitorGroups.js:
    • Update the label of the only Switch element to a translatable Display visitor groups in dashboard string.
    • Also update the string in the test file SettingsCardVisitorGroups.test.js.
  • Move the SettingsCardVisitorGroups.js and its associated story and test files to assets/js/modules/analytics-4/components/settings/AudienceSegmentation or assets/js/modules/analytics-4/components/audience-segmentation/settings/ if 8495 has been already merged.

Test Coverage

  • Tests should be updated as described above.

QA Brief

  • Go to Site Kit > Settings > Admin Settings. There should be Visitor groups panel with a toggle setting.
  • Label for toggle should read Display visitor groups in dashboard
  • Open the browser console and in network tab, notice that when we change the state of the toggle, it should send a request to google-site-kit/v1/modules/analytics-4/data/audience-settings. Request should be successful with 200 status code.

Changelog entry

  • Fix typo in the "Visitor groups" admin setting.
@techanvil techanvil added Module: Analytics Google Analytics module related issues P1 Medium priority Type: Enhancement Improvement of an existing feature labels Apr 8, 2024
@ivonac4 ivonac4 added the Squad 2 (Team M) Issues for Squad 2 label Apr 9, 2024
@ivonac4 ivonac4 added the Next Up Issues to prioritize for definition label Apr 16, 2024
@nfmohit nfmohit assigned nfmohit and unassigned nfmohit Apr 23, 2024
@kuasha420 kuasha420 assigned kuasha420 and unassigned kuasha420 Apr 23, 2024
@techanvil techanvil self-assigned this Apr 24, 2024
@techanvil
Copy link
Collaborator Author

IB ✅

@techanvil techanvil removed their assignment Apr 24, 2024
@ivonac4 ivonac4 removed the Next Up Issues to prioritize for definition label Apr 25, 2024
@ankitrox ankitrox self-assigned this Apr 29, 2024
@ankitrox ankitrox removed their assignment Apr 29, 2024
@nfmohit nfmohit assigned nfmohit and ankitrox and unassigned nfmohit Apr 29, 2024
@ankitrox ankitrox assigned nfmohit and unassigned ankitrox Apr 30, 2024
@nfmohit nfmohit removed their assignment May 2, 2024
@kelvinballoo kelvinballoo self-assigned this May 6, 2024
@kelvinballoo
Copy link
Collaborator

QA Update ✅

Tested this on develop on WP 6.5.2 PHP 8.0 and it's looking good:

  • Copy has been updated: There is no plural for the word 'visitor'
    Screenshot 2024-05-06 at 14 18 32
  • Toggling the setting on and off will trigger the required request with 200 status to:
    google-site-kit/v1/modules/analytics-4/data/audience-settings.
    Screenshot 2024-05-06 at 14 20 33

Moving ticket to Approved.

@kelvinballoo kelvinballoo removed their assignment May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: Analytics Google Analytics module related issues P1 Medium priority Squad 2 (Team M) Issues for Squad 2 Type: Enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

7 participants