jenkins-bot has submitted this change and it was merged.

Change subject: Adding support for mobile flag in EchoInteraction logging schema
......................................................................


Adding support for mobile flag in EchoInteraction logging schema

Change-Id: I8db8d3c4eb2bb396195c42073a95ac8530fe869d
Dependency: I3cb6b9cff91a927b03d30e5f3fec9984cf3f790b
---
M javascripts/modules/notifications/NotificationsOverlay.js
M javascripts/specials/notifications.js
2 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Jdlrobson: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/javascripts/modules/notifications/NotificationsOverlay.js 
b/javascripts/modules/notifications/NotificationsOverlay.js
index e584b20..9b5b99d 100644
--- a/javascripts/modules/notifications/NotificationsOverlay.js
+++ b/javascripts/modules/notifications/NotificationsOverlay.js
@@ -67,7 +67,7 @@
                                                                        } );
                                                                }
                                                                // Set up event 
logging for each notification
-                                                               
mw.echo.setupNotificationLogging( $notification, 'mobile-overlay' );
+                                                               
mw.echo.setupNotificationLogging( $notification, 'mobile-overlay', true );
                                                        } );
 
                                                        self.markAllAsRead();
diff --git a/javascripts/specials/notifications.js 
b/javascripts/specials/notifications.js
index 9ab3a09..e3e8915 100644
--- a/javascripts/specials/notifications.js
+++ b/javascripts/specials/notifications.js
@@ -20,7 +20,7 @@
                header = mw.config.get( 'wgEchoDateHeader' );
                mw.loader.using( 'ext.echo.base', function() {
                        $( '.mw-echo-notification' ).each( function () {
-                               mw.echo.setupNotificationLogging( $( this ), 
'mobile-archive' );
+                               mw.echo.setupNotificationLogging( $( this ), 
'mobile-archive', true );
                        } );
                } );
        }
@@ -67,7 +67,7 @@
                                        unread.push( id );
                                }
                                mw.loader.using( 'ext.echo.base', function() {
-                                       mw.echo.setupNotificationLogging( $li, 
'mobile-archive' );
+                                       mw.echo.setupNotificationLogging( $li, 
'mobile-archive', true );
                                } );
                        } );
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8db8d3c4eb2bb396195c42073a95ac8530fe869d
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Kaldari <rkald...@wikimedia.org>
Gerrit-Reviewer: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to