Skip to content

Conversation

@dskamiotis
Copy link
Contributor

What does this change?

Reduces the commercial-user-module-uid2 AB test audience size to 0% to enable opt-in only testing.

Why?

We're investigating why pbjs.getUserIds().uid2 returns undefined instead of a valid UID2 token as expected per The Trade Desk.

To debug this issue in production, we need to:

  1. Test the UID2 module without the storage property (commercial#2335)
  2. Limit exposure to opt-in users only (via ?ab-commercial-user-module-uid2=variant)
  3. Validate token generation in the production environment

Setting audienceSize: 0% allows us to safely test the module configuration with real production data while controlling exactly who is exposed to the test via query parameters.

Testing

  • Verify test can be activated via ?ab-commercial-user-module-uid2=variant
  • Confirm pbjs.getUserIds().uid2 returns a valid token with the updated configuration
  • Validate no users are automatically enrolled (0% audience)

@dskamiotis dskamiotis requested a review from a team as a code owner December 9, 2025 13:00
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@dskamiotis dskamiotis added the run_chromatic Runs chromatic when label is applied label Dec 9, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Dec 9, 2025
@Jakeii
Copy link
Member

Jakeii commented Dec 9, 2025

Ah this is a "holdback" test, so setting to 0% won't stop users getting this.

@dskamiotis
Copy link
Contributor Author

dskamiotis commented Dec 9, 2025

Ah this is a "holdback" test, so setting to 0% won't stop users getting this.

Even with the changes to the commercial PR here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants