Commit 55440ac9bec0b2e113d75540b898e4b46d766ba1
1 parent
525fa6c1
Exists in
spb-stable
and in
2 other branches
Make a comment to MR whne milestone was changed
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Showing
2 changed files
with
8 additions
and
0 deletions
Show diff stats
app/services/merge_requests/base_service.rb
| @@ -16,5 +16,9 @@ module MergeRequests | @@ -16,5 +16,9 @@ module MergeRequests | ||
| 16 | merge_request.project.execute_hooks(merge_request.to_hook_data, :merge_request_hooks) | 16 | merge_request.project.execute_hooks(merge_request.to_hook_data, :merge_request_hooks) |
| 17 | end | 17 | end |
| 18 | end | 18 | end |
| 19 | + | ||
| 20 | + def create_milestone_note(merge_request) | ||
| 21 | + Note.create_milestone_change_note(merge_request, merge_request.project, current_user, merge_request.milestone) | ||
| 22 | + end | ||
| 19 | end | 23 | end |
| 20 | end | 24 | end |
app/services/merge_requests/update_service.rb
| @@ -22,6 +22,10 @@ module MergeRequests | @@ -22,6 +22,10 @@ module MergeRequests | ||
| 22 | if params.present? && merge_request.update_attributes(params) | 22 | if params.present? && merge_request.update_attributes(params) |
| 23 | merge_request.reset_events_cache | 23 | merge_request.reset_events_cache |
| 24 | 24 | ||
| 25 | + if merge_request.previous_changes.include?('milestone_id') | ||
| 26 | + create_milestone_note(merge_request) | ||
| 27 | + end | ||
| 28 | + | ||
| 25 | if merge_request.previous_changes.include?('assignee_id') | 29 | if merge_request.previous_changes.include?('assignee_id') |
| 26 | notification_service.reassigned_merge_request(merge_request, current_user) | 30 | notification_service.reassigned_merge_request(merge_request, current_user) |
| 27 | create_assignee_note(merge_request) | 31 | create_assignee_note(merge_request) |