diff options
author | Douwe Maan <douwe@gitlab.com> | 2018-06-13 11:23:32 +0300 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2018-06-13 11:23:32 +0300 |
commit | 504f3620330fa0cc8b6f3cb3f1647db2158a036f (patch) | |
tree | 870f125ae32037fe5b5f6b5da8f1ec4f7c6fb444 /app/models | |
parent | 886b81e5c71529f1d61c4de4c16aab6171e1600e (diff) | |
parent | 9aad5ed0a23a6386bf48f166a6b07b3a41d9c2c1 (diff) |
Merge branch '42342-teams-pipeline-notifications' into 'master'
Resolve "Microsoft Teams notification for pipelines do not work"
Closes #42342
See merge request gitlab-org/gitlab-ce!19632
Diffstat (limited to 'app/models')
3 files changed, 7 insertions, 6 deletions
diff --git a/app/models/project_services/chat_message/base_message.rb b/app/models/project_services/chat_message/base_message.rb index 22a65b5145e..f710fa85b5d 100644 --- a/app/models/project_services/chat_message/base_message.rb +++ b/app/models/project_services/chat_message/base_message.rb @@ -26,13 +26,18 @@ module ChatMessage end end - def pretext + def summary return message if markdown format(message) end + def pretext + summary + end + def fallback + format(message) end def attachments diff --git a/app/models/project_services/chat_message/pipeline_message.rb b/app/models/project_services/chat_message/pipeline_message.rb index 2135122278a..96fd23aede3 100644 --- a/app/models/project_services/chat_message/pipeline_message.rb +++ b/app/models/project_services/chat_message/pipeline_message.rb @@ -23,10 +23,6 @@ module ChatMessage '' end - def fallback - format(message) - end - def attachments return message if markdown diff --git a/app/models/project_services/microsoft_teams_service.rb b/app/models/project_services/microsoft_teams_service.rb index 2facff53e26..99500caec0e 100644 --- a/app/models/project_services/microsoft_teams_service.rb +++ b/app/models/project_services/microsoft_teams_service.rb @@ -44,7 +44,7 @@ class MicrosoftTeamsService < ChatNotificationService def notify(message, opts) MicrosoftTeams::Notifier.new(webhook).ping( title: message.project_name, - pretext: message.pretext, + summary: message.summary, activity: message.activity, attachments: message.attachments ) |