Merge pull request #34 from torchbox/cookie-name-fix

Fix tracking cookie name to be consistent with worker and backend
wagtail-nest · Jul 8, 2021 · 286af8f · 286af8f
2 parents bc18eaf + 2322e06
commit 286af8f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/wagtail_ab_testing/static/wagtail_ab_testing/js/tracker.js b/wagtail_ab_testing/static/wagtail_ab_testing/js/tracker.js
@@ -25,7 +25,7 @@
     if (window.wagtailAbTesting) {
         // Register the user as a participant if they haven't registered yet
         if (window.wagtailAbTesting.testId) {
-            var cookieName = 'abtesting-' + window.wagtailAbTesting.testId + '-version';
+            var cookieName = 'wagtail-ab-testing_' + window.wagtailAbTesting.testId + '_version';
             if (!document.cookie.includes(cookieName)) {
                 fetch(
                     window.wagtailAbTesting.urls.registerParticipant,
@@ -88,7 +88,8 @@
                 }
 
                 goalsForEvent.forEach(function (testId) {
-                    var version = getCookie('abtesting-' + testId + '-version');
+                    var cookieName = 'wagtail-ab-testing_' + testId + '_version';
+                    var version = getCookie(cookieName);
 
                     if (version) {
                         fetch(