Skip to content

Format deployment message notification in GitLab for Slack app integration

Rob Rotaru requested to merge robert-rotaru/gitlab:robert-rotaru-master-patch-73174 into master

What does this MR do and why?

Update deployment_message.rb

Bug fix for #395640 (closed). Wrap GitLab for Slack deployment message text in format() to get correct syntax for Slack hyperlinks

Changelog: changed

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

** Note: I am not able to configure this myself due to locked down permissions on my Slack instance for installing GitLab for Slack from a local instance. I will need some help from GitLab to perform the validation steps below. I was able to validate all RSpec tests are passing.

  1. Visit any project's Integration settings and configure the GitLab for Slack app integration.
  2. Once configured, enable the "A deployment is started or finished" checkbox.
  3. Run a pipeline with a deployment and validate the Slack message output.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports