Users can request new treatment via query string
Bug: T204746 Change-Id: Icea989a401a5de8772fdcd9f25f100899bc1c6f3
This commit is contained in:
parent
2163d5f965
commit
2f3fed5f4a
|
@ -21,7 +21,10 @@
|
||||||
samplingRate: ( CURRENT_NS === NS_MAIN ) ? config.get( 'wgMinervaABSamplingRate', 0 ) : 0,
|
samplingRate: ( CURRENT_NS === NS_MAIN ) ? config.get( 'wgMinervaABSamplingRate', 0 ) : 0,
|
||||||
sessionId: user.sessionId()
|
sessionId: user.sessionId()
|
||||||
} ),
|
} ),
|
||||||
newTreatmentEnabled = abTest.isB();
|
QUERY_STRING_FLAG = mw.util.getParamValue( 'minerva-issues' ),
|
||||||
|
// Per T204746 a user can request the new treatment regardless of test group
|
||||||
|
isUserRequestingNewTreatment = QUERY_STRING_FLAG === 'b',
|
||||||
|
newTreatmentEnabled = abTest.isB() || isUserRequestingNewTreatment;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @typedef {Object} IssueSummary
|
* @typedef {Object} IssueSummary
|
||||||
|
@ -33,7 +36,7 @@
|
||||||
|
|
||||||
function isLoggingRequired( pageIssues ) {
|
function isLoggingRequired( pageIssues ) {
|
||||||
// No logging necessary when the A/B test is disabled (control group).
|
// No logging necessary when the A/B test is disabled (control group).
|
||||||
return abTest.isEnabled() && pageIssues.length;
|
return !isUserRequestingNewTreatment && abTest.isEnabled() && pageIssues.length;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in New Issue