Skip to content

Slack notifications - Respect "Notify only default branch" for push events

Mario de la Ossa requested to merge 33570-slack-notify-default-branch into master

What does this MR do?

As per #33570 (closed), makes ChatNotificationService (parent of SlackService and MattermostService) respect the notify_only_default_branch setting for push events, so that pushes to branches that are not the default do not get sent to Slack/etc.

Are there points in the code the reviewer needs to double check?

Should we also honor this setting for MergeRequest events?

Closes #33570 (closed)

Edited by Mario de la Ossa

Merge request reports