Jdlrobson has uploaded a new change for review.

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


Change subject: Story 1073: Enable campaigns on mobile
......................................................................

Story 1073: Enable campaigns on mobile

* Make sure login page passes campaign to sign up
* Add campaign keywords to all CTAs
Change-Id: Ia4b87c9a99d98c1ebf93704b0a9fd6c8670612a2
Dependency: Ia95d30bc58182910b4c2d3fc1da3960aa403562c
---
M includes/skins/UserLoginMobileTemplate.php
M javascripts/common/CtaDrawer.js
M javascripts/modules/editor/editor.js
M javascripts/modules/mf-watchstar.js
M javascripts/modules/uploads/PhotoUploader.js
5 files changed, 16 insertions(+), 2 deletions(-)


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

diff --git a/includes/skins/UserLoginMobileTemplate.php 
b/includes/skins/UserLoginMobileTemplate.php
index 25d473d..76f7338 100644
--- a/includes/skins/UserLoginMobileTemplate.php
+++ b/includes/skins/UserLoginMobileTemplate.php
@@ -30,6 +30,11 @@
                if ( isset( $actionQuery['returntoquery'] ) ) {
                        $query['returntoquery'] = $actionQuery['returntoquery'];
                }
+               // For Extension:Campaigns
+               $campaign = $this->getSkin()->getRequest()->getText( 'campaign' 
);
+               if ( $campaign ) {
+                       $query['campaign'] = $campaign;
+               }
 
                $signupLink = Linker::link( SpecialPage::getTitleFor( 
'Userlogin' ),
                        wfMessage( 'mobile-frontend-main-menu-account-create' 
)->text(),
diff --git a/javascripts/common/CtaDrawer.js b/javascripts/common/CtaDrawer.js
index 3ec84bf..42a3a81 100644
--- a/javascripts/common/CtaDrawer.js
+++ b/javascripts/common/CtaDrawer.js
@@ -9,11 +9,11 @@
                template: M.template.get( 'ctaDrawer' ),
 
                preRender: function( options ) {
-                       var params = {
+                       var params = $.extend( options.queryString || {}, {
                                // use wgPageName as this includes the 
namespace if outside Main
                                returnto: mw.config.get( 'wgPageName' ),
                                returntoquery: options.returnToQuery
-                       };
+                       } );
 
                        options.loginUrl = M.history.getArticleUrl( 
'Special:UserLogin', params );
                        options.signupUrl = M.history.getArticleUrl( 
'Special:UserLogin', $.extend( params, { type: 'signup' } ) );
diff --git a/javascripts/modules/editor/editor.js 
b/javascripts/modules/editor/editor.js
index b6bb274..4388448 100644
--- a/javascripts/modules/editor/editor.js
+++ b/javascripts/modules/editor/editor.js
@@ -7,6 +7,9 @@
                isEditingSupported = M.router.isSupported() && !blacklisted,
                CtaDrawer = M.require( 'CtaDrawer' ),
                drawer = new CtaDrawer( {
+                       queryString: {
+                               campaign: 'mobile_editPageActionCta'
+                       },
                        returnToQuery: 'article_action=edit',
                        content: mw.msg( 'mobile-frontend-editor-cta' )
                } );
diff --git a/javascripts/modules/mf-watchstar.js 
b/javascripts/modules/mf-watchstar.js
index fbabb6d..cd21e78 100644
--- a/javascripts/modules/mf-watchstar.js
+++ b/javascripts/modules/mf-watchstar.js
@@ -7,6 +7,9 @@
                CtaDrawer = M.require( 'CtaDrawer' ),
                drawer = new CtaDrawer( {
                        content: mw.msg( 'mobile-frontend-watchlist-cta' ),
+                       queryString: {
+                               campaign: 'mobile_watchPageActionCta'
+                       },
                        returnToQuery: 'article_action=watch'
                } );
 
diff --git a/javascripts/modules/uploads/PhotoUploader.js 
b/javascripts/modules/uploads/PhotoUploader.js
index 0fef3d5..ae72e8f 100644
--- a/javascripts/modules/uploads/PhotoUploader.js
+++ b/javascripts/modules/uploads/PhotoUploader.js
@@ -110,6 +110,9 @@
                        if ( !M.isLoggedIn() ) {
                                ctaDrawer = new CtaDrawer( {
                                        content: mw.msg( 
'mobile-frontend-photo-upload-cta' ),
+                                       queryString: {
+                                               campaign: 
'mobile_uploadPageActionCta'
+                                       },
                                        returnToQuery: 
'article_action=photo-upload'
                                } );
                                this.$el.click( function( ev ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia4b87c9a99d98c1ebf93704b0a9fd6c8670612a2
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