EBernhardson has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/376858 )

Change subject: Only show human relevance survey to a browser every 2 days
......................................................................

Only show human relevance survey to a browser every 2 days

Some users, especially if they visit a page with a low sampling
rate multiple times, will see the survey more often than expected
and potentially be annoyed.

Setup a timeout such that there will be a 2 day delay between
occurances of a user being shown the survey.

Change-Id: Ic3da1c68b09b7461db10d11e2cbd9bed3bca9143
(cherry picked from commit 198c8a968114e45ea01902bf84d7dd69d00f7b76)
---
M modules/ext.wikimediaEvents.humanSearchRelevance.js
1 file changed, 15 insertions(+), 4 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikimediaEvents 
refs/changes/58/376858/1

diff --git a/modules/ext.wikimediaEvents.humanSearchRelevance.js 
b/modules/ext.wikimediaEvents.humanSearchRelevance.js
index 5ee8fb2..47d7591 100644
--- a/modules/ext.wikimediaEvents.humanSearchRelevance.js
+++ b/modules/ext.wikimediaEvents.humanSearchRelevance.js
@@ -24,10 +24,13 @@
                return;
        }
 
-       // This value is coded into the page output and cached in varnish. That
-       // means any changes to sampling rates or pages chosen will take up to a
-       // week to propogate into the wild.
-       var config = mw.config.get( 'wgWMESearchRelevancePages' );
+       // The config value is coded into the page output and cached in varnish.
+       // That means any changes to sampling rates or pages chosen will take 
up to
+       // a week to propogate into the wild.
+       var timeout,
+               config = mw.config.get( 'wgWMESearchRelevancePages' ),
+               timeoutKey = 'wme-humrel-timeout',
+               now = new Date().getTime();
 
        // bad configuration
        if ( !config.hasOwnProperty( 'sampleRate' ) || !config.hasOwnProperty( 
'queries' ) ) {
@@ -39,6 +42,14 @@
                return;
        }
 
+       timeout = mw.storage.get( timeoutKey );
+       if ( timeout && timeout > now ) {
+               // User has seen the survey recently
+               return;
+       }
+       // Don't show the survey to same browser for 2 days, to prevent 
annoying users
+       mw.storage.set( timeoutKey, now + 2 * 86400 );
+
        function askQuestion() {
                mw.loader.using( [
                        'oojs-ui-core',

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic3da1c68b09b7461db10d11e2cbd9bed3bca9143
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikimediaEvents
Gerrit-Branch: wmf/1.30.0-wmf.16
Gerrit-Owner: EBernhardson <ebernhard...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to