Auxia control group#14775
Merged
Merged
Conversation
8f9b5a9 to
82f3a68
Compare
|
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
juabara
approved these changes
Nov 3, 2025
shtukas
approved these changes
Nov 3, 2025
Contributor
|
Nice solution to the problem, and hopefully at some point, we will not need to set the mvt value anymore 🙂 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The new requirement:
The proposed solution:
NoAuxiaSignInGateAB test. This means the page tracks membership of the "test" through ophan. There are no variants in this test, it's just a way to track these browsers.NoAuxiaSignInGatetest, and it excludes them from Auxia based on this.NoAuxiaSignInGate5% group using the existing ab_test_array field.Note - this is separate from the existing logic in SDC that splits the audience between Auxia and the hardcoded gate logic. It's necessary to do this control group client-side so that we can track it in the pageview data.
Implementation
isInAuxiaControlGroup, to the SDC endpoint for sign-in gatesNoAuxiaSignInGateAB test. This isn't a real AB test, and it only has a 'control' variant. It's a way to select a fixed 5% of browsers for the auxia control group.canShowSignInGatePortalfunction to make the parameters clearer. It has a lot of parameters, so it now takes an interfaceCanShowSignInGateProps, which makes calling it cleaner and safer.There will need to be a change in SDC to exclude users from Auxia if this flag is set.
Testing
Tested locally by changing the mvt cookie and observing the request to SDC.
With an mvt value in the bottom 95% of the range, the flag is set to false:

With an mvt value in the top 5% of the range, the flag is set to true:

The

abTestRegisterevent that we send to ophan includes the newNoAuxiaSignInGate"test":Note - the existing
SignInGateMainControltest here is meaningless and needs to be removed!