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

Change subject: Turn on email bundling for post edit/reply + instructions
......................................................................


Turn on email bundling for post edit/reply + instructions

To test this on a local environment, this will need:

1. set the time interval in $wgEchoBundleEmailInterval between sending
   emails, by default it's zero and disable email bundling

2. Config with a redis job queue, I think only redis support
   delayed job queue

3. config the bundle email job with something like:
   $wgJobTypeConf['MWEchoNotificationEmailBundleJob'] = array(
        'class'       => 'JobQueueRedis',
        'redisServer' => '127.0.0.1',
        'redisConfig' => array( 'connectTimeout' => 1 ),
        'claimTTL'    => 3600,
        'checkDelay'  => true
  );

Change-Id: I76d08706cdd90fe0cd2fd745c36bf50f9b7672d3
---
M includes/Notifications/Notifications.php
1 file changed, 2 insertions(+), 2 deletions(-)

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



diff --git a/includes/Notifications/Notifications.php 
b/includes/Notifications/Notifications.php
index 981b37f..699ee2c 100644
--- a/includes/Notifications/Notifications.php
+++ b/includes/Notifications/Notifications.php
@@ -38,7 +38,7 @@
                'title-params' => array( 'agent', 'subject', 'flow-title', 
'title', 'post-permalink' ),
                'bundle' => array(
                        'web' => true,
-                       'email' => false
+                       'email' => true,
                ),
                'bundle-message' => 'flow-notification-reply-bundle',
                'bundle-params' => array( 'agent', 'subject', 'title', 
'post-permalink', 'agent-other-display', 'agent-other-count' ),
@@ -58,7 +58,7 @@
                'title-params' => array( 'agent', 'subject', 'flow-title', 
'title', 'post-permalink' ),
                'bundle' => array(
                        'web' => true,
-                       'email' => false
+                       'email' => true,
                ),
                'bundle-message' => 'flow-notification-edit-bundle',
                'bundle-params' => array( 'agent', 'subject', 'title', 
'post-permalink', 'agent-other-display', 'agent-other-count' ),

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I76d08706cdd90fe0cd2fd745c36bf50f9b7672d3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
Gerrit-Reviewer: EBernhardson <[email protected]>
Gerrit-Reviewer: SG <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to