-
Notifications
You must be signed in to change notification settings - Fork 13
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
Omit users on the Contributions Only checkout from the coverTransactionCost test #6429
Conversation
Size Change: +9 B (0%) Total Size: 2.28 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, sorry I missed that, thanks George.
It's OK, we had it set up like this on the original test too but it wasn't an issue for that test as we'd positioned the cover transaction component at the bottom only, so it appeared correctly for the |
Seen on PROD (merged by @GHaberis 10 minutes and 54 seconds ago)
Sentry Release: support-client-side, support |
What are you doing in this PR?
We have a "Contributions Only" checkout flow for some countries: eg. https://support.theguardian.com/int/contribute?country=OM
We shouldn't be bucketing users on this "Contributions Only" checkout flow in the
coverTransactionCost
A/B/C test because this checkout doesn't show the one-time choice cards component on the checkout page, which means we users allocated tovariantB
will never see the cover transaction component as it sits in this component.