Jdlrobson has uploaded a new change for review.
https://gerrit.wikimedia.org/r/155675
Change subject: Setup A/B test for editing / performance
......................................................................
Setup A/B test for editing / performance
Does reloading page via JavaScript after edit encourage more editing?
Theoretically, re-rendering a page via JavaScript is quicker, but this
can vary depending on internet connection, size of page and various other
factors.
It is possible a slower editing experience might deter users from editing
further but we have no data to say one way or another.
This buckets our users into 2 groups
Bucket A: After editing the page will be reloaded via HTTP request
Bucket B: After editing the page will be re-rendered via JavaScript.
The purpose of the A/B test is to see if there are significantly more
edits under one bucket that another, or if there is no noticeable change
in behaviour.
S# Please enter the commit message for your changes. Lines starting
Change-Id: I36208c0d6964685953a6bb2fc32eb7d15fe70834
---
M javascripts/modules/editor/EditorOverlayBase.js
1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/75/155675/1
diff --git a/javascripts/modules/editor/EditorOverlayBase.js
b/javascripts/modules/editor/EditorOverlayBase.js
index bc650fa..3d1473d 100644
--- a/javascripts/modules/editor/EditorOverlayBase.js
+++ b/javascripts/modules/editor/EditorOverlayBase.js
@@ -4,6 +4,7 @@
schema = M.require( 'loggingSchemas/mobileWebEditing' ),
toast = M.require( 'toast' ),
user = M.require( 'user' ),
+ isTestA = M.isTestA,
EditorOverlayBase;
/**
@@ -83,6 +84,14 @@
// FIXME: use generic method for following 3 lines
M.pageApi.invalidatePage( title );
+
+ // FIXME: Remove when A/B test has run its course.
+ // Does reloading page via JavaScript after edit
encourage more editing?
+ if ( isTestA ) {
+ window.location = mw.util.getUrl( title );
+ return;
+ }
+
new Page( { title: title, el: $( '#content_wrapper' ) }
).on( 'ready', M.reloadPage ).
on( 'error', function() {
// Force refresh when something goes
wrong (see bug 62175 for example)
--
To view, visit https://gerrit.wikimedia.org/r/155675
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I36208c0d6964685953a6bb2fc32eb7d15fe70834
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
