Skip to content

GOOWOO-677: Add Tracking to Notifications System#3594

Merged
asvinb merged 4 commits into
feature/notifications-systemfrom
add/GOOWOO-677-tracking
Jul 16, 2026
Merged

GOOWOO-677: Add Tracking to Notifications System#3594
asvinb merged 4 commits into
feature/notifications-systemfrom
add/GOOWOO-677-tracking

Conversation

@asvinb

@asvinb asvinb commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Changes proposed in this Pull Request:

Closes https://linear.app/a8c/issue/GOOWOO-677/fe-notification-tracking-analytics

Replace this with a good description of your changes & reasoning.

Screenshots:

Detailed test instructions:

  1. Confirm tracking is allowed in WooCommerce > Settings > Advanced > Woo.com
  2. Run command localStorage.setItem( 'debug', 'wc-admin:*' ) in console.
  3. Make sure that events are triggered properly with the properties mentioned in AC
  4. Clicking through a notification CTA will append the following parameters to the URL:
    • referrer_type: notification
    • referrer_id:
  5. Any events triggered when the above URL params are present will be passed along the tracking data. For e.g when creating a new campaign, the wcadmin_gla_launch_paid_campaign_button_click event will now emit the referrer_id and referrer_type properties.

Additional details:

Changelog entry

@github-actions github-actions Bot added type: enhancement The issue is a request for an enhancement. changelog: add A new feature, function, or functionality was added. labels Jul 15, 2026
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.4%. Comparing base (36790cc) to head (191feea).
⚠️ Report is 242 commits behind head on feature/notifications-system.

Additional details and impacted files

Impacted file tree graph

@@                        Coverage Diff                        @@
##             feature/notifications-system   #3594      +/-   ##
=================================================================
- Coverage                            67.2%   64.4%    -2.8%     
=================================================================
  Files                                 944     400     -544     
  Lines                               29079    6890   -22189     
  Branches                             1738    1740       +2     
=================================================================
- Hits                                19535    4438   -15097     
+ Misses                               9282    2190    -7092     
  Partials                              262     262              
Flag Coverage Δ
js-unit-tests 64.4% <100.0%> (+0.1%) ⬆️
php-unit-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...tions-slot/components/notifications-panel/index.js 100.0% <100.0%> (ø)
js/src/utils/tracks.js 79.5% <100.0%> (+1.7%) ⬆️

... and 551 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@asvinb
asvinb merged commit 370f209 into feature/notifications-system Jul 16, 2026
5 checks passed
@asvinb
asvinb deleted the add/GOOWOO-677-tracking branch July 16, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: add A new feature, function, or functionality was added. type: enhancement The issue is a request for an enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants