support-frontend icon indicating copy to clipboard operation
support-frontend copied to clipboard

Persist landing page test tracking

Open tomrf1 opened this issue 1 year ago â€ĸ 5 comments

Currently, if an AB test is targeted specifically at the landing page then the tracking doesn't carry through to the checkout. This means it doesn't get included in the acquisition event.

This PR fixes this by changing the landing page to add the AB test participations to session storage. Subsequent pages will then pick up participations from session storage.

tomrf1 avatar Oct 07 '24 09:10 tomrf1

Size Change: +1.71 kB (+0.07%)

Total Size: 2.29 MB

â„šī¸ View Unchanged
Filename Size Change
./public/compiled-assets/javascripts/[countryGroupId]/events/router.js 112 kB +104 B (+0.09%)
./public/compiled-assets/javascripts/[countryGroupId]/router.js 258 kB +91 B (+0.04%)
./public/compiled-assets/javascripts/ausMomentMap.js 108 kB -1 B (0%)
./public/compiled-assets/javascripts/contributionsRedirectStyles.js 20 B 0 B
./public/compiled-assets/javascripts/digitalSubscriptionLandingPage.js 242 kB +109 B (+0.05%)
./public/compiled-assets/javascripts/downForMaintenancePage.js 69.2 kB +103 B (+0.15%)
./public/compiled-assets/javascripts/error404Page.js 69.2 kB +101 B (+0.15%)
./public/compiled-assets/javascripts/error500Page.js 69.1 kB +102 B (+0.15%)
./public/compiled-assets/javascripts/favicons.js 617 B 0 B
./public/compiled-assets/javascripts/paperSubscriptionCheckoutPage.js 195 kB +102 B (+0.05%)
./public/compiled-assets/javascripts/paperSubscriptionLandingPage.js 85.7 kB +92 B (+0.11%)
./public/compiled-assets/javascripts/payPalErrorPage.js 67.5 kB +112 B (+0.17%)
./public/compiled-assets/javascripts/payPalErrorPageStyles.js 20 B 0 B
./public/compiled-assets/javascripts/promotionTerms.js 72.1 kB +99 B (+0.14%)
./public/compiled-assets/javascripts/subscriptionsLandingPage.js 71.3 kB +101 B (+0.14%)
./public/compiled-assets/javascripts/subscriptionsRedemptionPage.js 126 kB +98 B (+0.08%)
./public/compiled-assets/javascripts/supporterPlusLandingPage.js 311 kB +184 B (+0.06%)
./public/compiled-assets/javascripts/unsupportedBrowserStyles.js 20 B 0 B
./public/compiled-assets/javascripts/weeklySubscriptionCheckoutPage.js 192 kB +100 B (+0.05%)
./public/compiled-assets/javascripts/weeklySubscriptionLandingPage.js 85.6 kB +111 B (+0.13%)
./public/compiled-assets/webpack/136.js 2.17 kB 0 B
./public/compiled-assets/webpack/187.js 21.4 kB +105 B (+0.49%)
./public/compiled-assets/webpack/344.js 2.01 kB 0 B
./public/compiled-assets/webpack/671.js 21.8 kB 0 B
./public/compiled-assets/webpack/706.js 107 kB 0 B

compressed-size-action

github-actions[bot] avatar Oct 07 '24 09:10 github-actions[bot]

Why would we not just target the landing page and checkout page?

rupertbates avatar Oct 07 '24 10:10 rupertbates

Why would we not just target the landing page and checkout page?

we don't want people who go straight to checkout to be included in the test - they must go via the landing page. We're sending quite a lot of users straight to checkout now. The existing setup for AB test targeting doesn't really support the 2-step flow correctly

tomrf1 avatar Oct 07 '24 10:10 tomrf1

DP have approved the use of the session storage item

tomrf1 avatar Oct 15 '24 12:10 tomrf1

DP have approved the use of the session storage item

Excellent, this looks good to go for me then

GHaberis avatar Oct 16 '24 10:10 GHaberis

Seen on PROD (merged by @tomrf1 12 minutes and 11 seconds ago)

Sentry Release: support-client-side, support

prout-bot avatar Oct 22 '24 07:10 prout-bot