Skip to content

Update Slack notifications integration UI text

Marcel Amirault requested to merge ui-text-slack-integration into master

What does this MR do?

Resolves #300622 (closed)

Updates the UI Text for the Slack integration. Only a few tweaks were needed as the majority of the fields were updated in !58570 (merged) (shared between all chat notification integrations). I only took screenshots of the top of the integration, as everything else is unchanged (shared with the Mattermost integration, so fixed in that MR. MR)

Screenshots (strongly suggested)

  • Before: Screen_Shot_2021-04-08_at_13.15.34
  • After: Screen_Shot_2021-04-08_at_13.15.21
  • Before: Screen_Shot_2021-04-08_at_13.14.54
  • After: Screen_Shot_2021-04-08_at_13.11.10

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Marcin Sedlak-Jakubowski

Merge request reports

Loading