From c3533deda1501accee8ba956539553e9af1f7e23 Mon Sep 17 00:00:00 2001 From: Marin Jankovski <marin@gitlab.com> Date: Thu, 3 Jul 2014 10:26:47 +0200 Subject: [PATCH] Don't notify the user that triggered the mr notification. --- app/services/notification_service.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index af61a94a175..c32ece50885 100644 --- a/app/services/notification_service.rb +++ b/app/services/notification_service.rb @@ -89,7 +89,7 @@ class NotificationService def merge_mr(merge_request, current_user) recipients = reject_muted_users([merge_request.author, merge_request.assignee], merge_request.target_project) recipients = recipients.concat(project_watchers(merge_request.target_project)).uniq - recipients.delete(merge_request.merge_event.author) if merge_request.merged? + recipients.delete(current_user) recipients.each do |recipient| mailer.merged_merge_request_email(recipient.id, merge_request.id, current_user.id) -- GitLab