wikimedia/mediawiki-skins-MinervaNeue
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/skins/MinervaNeue synced 2024-11-24 22:25:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Users can request new treatment via query string"

Browse source
This commit is contained in:
jenkins-bot 2018-09-20 19:02:59 +00:00 committed by Gerrit Code Review
parent 01dc9de92e 2f3fed5f4a
commit 1ad76e7d13
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
resources/skins.minerva.scripts/pageIssues.js
View file

@ -21,7 +21,10 @@
samplingRate: ( CURRENT_NS === NS_MAIN ) ? config.get( 'wgMinervaABSamplingRate', 0 ) : 0,
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
@ -33,7 +36,7 @@
function isLoggingRequired( pageIssues ) {
// No logging necessary when the A/B test is disabled (control group).
return abTest.isEnabled() && pageIssues.length;
return !isUserRequestingNewTreatment && abTest.isEnabled() && pageIssues.length;
}
/**