Jdlrobson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/73331


Change subject: Update to latest schema
......................................................................

Update to latest schema

Will log namespace as requested in Bug 51061
As well as user edit count at time of edit to identify new editors
As well as whether AB test was run

Dependency: I7e7a99fd5e17dc1b5038b14df21d32320e581806
Change-Id: I37079a3501844cff433ccadd42c0b813b7cc2e37
---
M javascripts/modules/editor/EditorOverlay.js
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/31/73331/1

diff --git a/javascripts/modules/editor/EditorOverlay.js 
b/javascripts/modules/editor/EditorOverlay.js
index 56ad1c7..94db84d 100644
--- a/javascripts/modules/editor/EditorOverlay.js
+++ b/javascripts/modules/editor/EditorOverlay.js
@@ -1,6 +1,7 @@
 ( function( M, $ ) {
 
        var Overlay = M.require( 'Overlay' ),
+               isTestA = Math.random() < 0.5,
                Page = M.require( 'page' ),
                popup = M.require( 'notifications' ),
                api = M.require( 'api' ),
@@ -30,6 +31,9 @@
                                        token: M.getSessionId(),
                                        action: action,
                                        section: this.sectionId,
+                                       namespace: mw.config.get( 
'wgNamespaceNumber' ),
+                                       userEditCount: mw.config.get( 
'wgUserEditCount' ),
+                                       isTestA: isTestA,
                                        pageId: mw.config.get( 'wgArticleId' ),
                                        username: mw.config.get( 'wgUserName' ),
                                        mobileMode: mw.config.get( 'wgMFMode' ),
@@ -200,7 +204,7 @@
                                        new Page( { title: title, el: $( 
'#content_wrapper' ) } );
                                        M.router.navigate( '' );
                                        self.hide();
-                                       if ( mw.config.get( 'wgUserEditCount' ) 
=== '0' ) {
+                                       if ( isTestA && mw.config.get( 
'wgUserEditCount' ) === '0' ) {
                                                msg = 
'mobile-frontend-editor-success-landmark-1';
                                        } else {
                                                className = 'toast';

-- 
To view, visit https://gerrit.wikimedia.org/r/73331
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I37079a3501844cff433ccadd42c0b813b7cc2e37
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to