Zero is a sensible default for this feature flag

Bug: T205008
Depends-On: I3640b28c6a930f6e934d22c3375bffcd539902aa
Change-Id: Ie5d63e265469219063e36d6e398fd0a0b1e864db
This commit is contained in:
jdlrobson 2018-11-26 14:12:26 -08:00 committed by Sam Smith
parent a15119bdc1
commit 5c40ddc477
2 changed files with 2 additions and 3 deletions

View File

@ -21,8 +21,7 @@ mw.loader.using( [
*/ */
schemaMobileWebMainMenuClickTracking = new Schema( schemaMobileWebMainMenuClickTracking = new Schema(
'MobileWebMainMenuClickTracking', 'MobileWebMainMenuClickTracking',
// todo: use a default value of 0 once config lands in production (T205008). mw.config.get( 'wgMinervaSchemaMainMenuClickTrackingSampleRate' ),
mw.config.get( 'wgMinervaSchemaMainMenuClickTrackingSampleRate', 0.5 ),
/** /**
* @property {Object} defaults Default options hash. * @property {Object} defaults Default options hash.
* @property {string} defaults.mobileMode whether user is in stable or beta * @property {string} defaults.mobileMode whether user is in stable or beta

View File

@ -51,7 +51,7 @@
"base": false, "base": false,
"beta": true "beta": true
}, },
"MinervaSchemaMainMenuClickTrackingSampleRate": 0.5, "MinervaSchemaMainMenuClickTrackingSampleRate": 0,
"MinervaABSamplingRate": 0 "MinervaABSamplingRate": 0
}, },
"ValidSkinNames": { "ValidSkinNames": {