diff options
| author | Sean McGivern <sean@mcgivern.me.uk> | 2017-10-12 14:39:51 +0000 |
|---|---|---|
| committer | Sean McGivern <sean@mcgivern.me.uk> | 2017-10-12 14:39:51 +0000 |
| commit | ca06ff27874d96705b7878a17179ffb25d809aba (patch) | |
| tree | b7722e47f160ffcb9b7f41e4a74017c930d460fb /app | |
| parent | f383220674a40e2c542ba03e88c4296c6042d1f1 (diff) | |
| parent | 486da72f2801cae0a170e4e4eafc4b928cd3f060 (diff) | |
| download | gitlab-ce-ca06ff27874d96705b7878a17179ffb25d809aba.tar.gz | |
Merge branch '37691-subscription-fires-multiple-notifications' into 'master'
fix multiple notifications from being sent for multiple labels
Closes #37691
See merge request gitlab-org/gitlab-ce!14798
Diffstat (limited to 'app')
| -rw-r--r-- | app/services/notification_service.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index 8d5da459882..be3b4b2ba07 100644 --- a/app/services/notification_service.rb +++ b/app/services/notification_service.rb @@ -390,7 +390,7 @@ class NotificationService end def relabeled_resource_email(target, labels, current_user, method) - recipients = labels.flat_map { |l| l.subscribers(target.project) } + recipients = labels.flat_map { |l| l.subscribers(target.project) }.uniq recipients = notifiable_users( recipients, :subscription, target: target, |