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

Integrate <ConversionTrackingToggle /> Component Into Ads and GA4 Module <SettingsView /> Components #8617

Closed
2 tasks
10upsimon opened this issue Apr 25, 2024 · 4 comments
Labels
javascript Pull requests that update Javascript code Module: Ads Google Ads module related issues Module: Analytics Google Analytics module related issues P0 High priority Team S Issues for Squad 1 Type: Enhancement Improvement of an existing feature

Comments

@10upsimon
Copy link
Collaborator

10upsimon commented Apr 25, 2024

Feature Description

With the Conversion Tracking UI toggle component and associated settings/datastore now complete (see #8616), the toggle is ready to be integrated into both the settings area of the Ads module, as well as the GA4 module.

This issue focusses on the read only / view settings area of said modules, i.e the <SettingsView /> component of each module.

The enabled/disabled status for Conversion Tracking should be displayed in read-only form based on the state of the toggle (i.e if conversion tracking enabled (true) via the applicable datastore selector).

Please reference the "Conversion Event Tracking Toggle & Settings" area of the design doc for more detailed information.


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

Acceptance criteria

  • The SettingsView component of Ads and Analytics modules should display the Conversion tracking label and value
  • Value should be sourced from Conversion Tracking datastore and displayed as Enabled or Disabled text
  • It should match figma designs for both modules

Implementation Brief

  • Update assets/js/modules/ads/components/settings/SettingsView.js:
    • Use the selector added in Create conversion-tracking.js Datastore #8615 to pull the conversion tracking enabled setting value
    • Use DisplaySetting component to render Enabled or Disabled text when adsPax feature flag is enabled
    • Use Enhanced conversion tracking for label
    • Follow the figma design
  • Update assets/js/modules/analytics-4/components/settings/SettingsView.js
    • Do the same as in Ads module settings view component
    • Follow the figma design

Test Coverage

  • Update VRT images

QA Brief

  • Setup Site Kit with conversionInfra feature flag
  • Configure Ads and GA4
  • Go to the Ads setting screen (Site Kit > Settings > Ads) and paste the following snippet into the browser console to enable Conversion tracking:
    • await googlesitekit.api.set( 'core', 'site', 'conversion-tracking', { settings: { enabled: true } } );
  • Reload the page
  • Validate that the Enhanced conversion tracking setting label is present with a value of "Enabled" as per the screenshot below:
image
  • Change the setting to be disabled using the following snippet in console: await googlesitekit.api.set( 'core', 'site', 'conversion-tracking', { settings: { enabled: false } } );
  • Reload the page
  • Validate that the Enhanced conversion tracking setting label is present with a value of "Enabled" as per the screenshot below:
image
  • Perform the same steps for the GA4 settings view

Changelog entry

  • Add "Enhanced Conversion Tracking" to settings views for Ads and GA4.
@10upsimon 10upsimon added javascript Pull requests that update Javascript code Module: Ads Google Ads module related issues Module: Analytics Google Analytics module related issues Team S Issues for Squad 1 Type: Enhancement Improvement of an existing feature labels Apr 25, 2024
@eclarke1 eclarke1 added the Next Up Issues to prioritize for definition label Apr 30, 2024
@zutigrm zutigrm assigned zutigrm and unassigned zutigrm Apr 30, 2024
@10upsimon 10upsimon assigned 10upsimon and unassigned 10upsimon May 3, 2024
@10upsimon
Copy link
Collaborator Author

AC ✅

@mxbclang mxbclang added the P0 High priority label May 6, 2024
@zutigrm zutigrm assigned zutigrm and unassigned zutigrm May 7, 2024
@10upsimon 10upsimon self-assigned this May 8, 2024
@10upsimon
Copy link
Collaborator Author

IB ✅

@10upsimon 10upsimon removed their assignment May 8, 2024
@benbowler benbowler assigned benbowler and unassigned benbowler May 9, 2024
@10upsimon 10upsimon self-assigned this May 14, 2024
@eclarke1 eclarke1 removed the Next Up Issues to prioritize for definition label May 14, 2024
@10upsimon 10upsimon removed their assignment May 22, 2024
@zutigrm zutigrm self-assigned this May 22, 2024
@10upsimon 10upsimon assigned 10upsimon and zutigrm and unassigned zutigrm and 10upsimon May 22, 2024
@zutigrm zutigrm assigned 10upsimon and unassigned zutigrm May 22, 2024
@10upsimon 10upsimon assigned zutigrm and unassigned 10upsimon May 22, 2024
@zutigrm zutigrm removed their assignment May 22, 2024
@benbowler benbowler assigned benbowler and 10upsimon and unassigned benbowler May 31, 2024
@10upsimon 10upsimon assigned benbowler and unassigned 10upsimon May 31, 2024
@benbowler benbowler removed their assignment May 31, 2024
@aaemnnosttv aaemnnosttv self-assigned this May 31, 2024
@aaemnnosttv
Copy link
Collaborator

Note for QA: I've made a few changes which don't match the designs above but, these were already inconsistent relative to the current state there (mostly for GA). I've left a note in Figma too, so we should mostly be reviewing this visually for consistency with the rest of the settings.

@aaemnnosttv aaemnnosttv removed their assignment May 31, 2024
@mohitwp
Copy link
Collaborator

mohitwp commented Jun 5, 2024

QA Update ✅

  • Tested on dev environment.
  • Figma have different design and current is more aligned with existing styles.
  • Verified toggled settings for both Analytics and Ads module.
  • Verified the SettingsView component of Ads and Analytics modules display the Conversion tracking label and value.
  • Verified value are sourced from Conversion Tracking datastore and displayed as Enabled or Disabled text.

Ads module

Recording.1038.mp4
Recording.1037.mp4

Analytics module

Recording.1039.mp4

@mohitwp mohitwp removed their assignment Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code Module: Ads Google Ads module related issues Module: Analytics Google Analytics module related issues P0 High priority Team S Issues for Squad 1 Type: Enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

8 participants