diff options
author | YarNayar <YarTheGreat@gmail.com> | 2017-07-25 14:56:09 +0300 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2018-03-26 15:24:52 +0300 |
commit | 99b01e23598e6b0b2bcae891939ea28c67f7b2e9 (patch) | |
tree | c32d351419cd399dcb104eac54563b7028bbd1d8 /app/services/merge_requests | |
parent | bb9d360c0a7daed6aa08a0635e084c314c2c8b3e (diff) |
Send notification emails when push to a merge request
Closes #23460
Diffstat (limited to 'app/services/merge_requests')
-rw-r--r-- | app/services/merge_requests/refresh_service.rb | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/app/services/merge_requests/refresh_service.rb b/app/services/merge_requests/refresh_service.rb index 18c40ce8992..1fb1796b56c 100644 --- a/app/services/merge_requests/refresh_service.rb +++ b/app/services/merge_requests/refresh_service.rb @@ -21,7 +21,7 @@ module MergeRequests comment_mr_branch_presence_changed end - comment_mr_with_commits + notify_about_push mark_mr_as_wip_from_commits execute_mr_web_hooks @@ -141,8 +141,8 @@ module MergeRequests end end - # Add comment about pushing new commits to merge requests - def comment_mr_with_commits + # Add comment about pushing new commits to merge requests and send nofitication emails + def notify_about_push return unless @commits.present? merge_requests_for_source_branch.each do |merge_request| @@ -155,6 +155,8 @@ module MergeRequests SystemNoteService.add_commits(merge_request, merge_request.project, @current_user, new_commits, existing_commits, @oldrev) + + notification_service.push_to_merge_request(merge_request, @current_user, new_commits: new_commits, existing_commits: existing_commits) end end |