Merge pull request #204 from spring-media/TRAC-1966_getpocket_referrer

changed setup t.co to t.co/
spring-media · Feb 6, 2024 · dcd77a4 · dcd77a4
2 parents 4d67a5f + 7aee179
commit dcd77a4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/extensions/doPlugins_global.js b/extensions/doPlugins_global.js
@@ -599,7 +599,7 @@ s._setExternalReferringDomainEvents = function (s) {
             channelCategory: 'Youtube',
         },
         {
-            matchRegex:  /^t.co|twitter.com|android-app:\/\/com.twitter.android/,
+            domains:  ['t.co/', 'twitter.com', 'com.twitter.android'],
             event: 'event51,event222',
             channel: 'Organic Social',
             channelCategory: 'Twitter',

diff --git a/tests/doplugins/doplugins_externel_referring_domain.test.js b/tests/doplugins/doplugins_externel_referring_domain.test.js
@@ -115,7 +115,7 @@ describe('External referring domains', () => {
         expect(addEventMock).toHaveBeenCalledWith('event51,event222');
     });
 
-    it('should set event51 if the referring domain includes t.co', () => {
+    it('should set event51 if the referring domain includes t.co/', () => {
         getReferrerMock.mockReturnValue('t.co/');
 
         s._setExternalReferringDomainEvents(s);