Repository navigation
Subscriptions: use experiment cohort for performance optimized paywalls - #9914
Merged
Merged
Conversation
Collaborator
Author
|
NOTE: this is a DRAFT. I am updating the description with testing steps shortly. :) |
performanceOptimizedPaywalls becomes a control/treatment experiment rather than a plain rollout. Treatment opens the faster paywall page, control the legacy one, and both carry experiment_perfpaywall=<cohort> so the frontend can compare the two cohorts directly. Users who could never see the faster page (no offers, entry points without one) are not enrolled. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Pick the cohort with the same enroll-then-when shape the onboarding experiment managers use, and name the enum PerformanceOptimizedPaywallsCohorts to match the plural Cohorts convention. An incoming experiment_perfpaywall param is now replaced rather than duplicated, on both the treatment and control paths, the same way trial and pir are replaced. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
shakyShane
force-pushed
the
sosbourne/perf-paywall-cohort
branch
from
October 1, 2026 14:57
8cd46bd to
3e10dcd
Compare
shakyShane
marked this pull request as ready for review
October 1, 2026 15:34
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
Contributor
|
@claude review |
Kyriakos-Georgiopoulos
approved these changes
Oct 2, 2026
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.


Task/Issue URL: https://app.asana.com/1/137249556945/task/1219056538070932
Related Privacy Config PR: duckduckgo/privacy-configuration#6002
Tech Design URL (if applicable):
API Proposals URL(s) (if applicable):
Description
Steps to test this PR
Unsupported - this proves the change makes no difference to old app users
/subscriptions?featurePage=duckaiexperiment_perfpaywallSupported - this proves the change will take effect when
minSupportedVersionis eventually used in the remote configI used a version of the remote config PR with this patch
tap 'VPN' or other entry point
verify the URL used is the new format like
/subscriptions/new/mobile/duckaiverify it DOES include the experiment param
experiment_perfpaywall=control|treatment