Skip to content

Exclude code owner from merge request participants to avoid spam

Mark Chao requested to merge 8396-exclude-code-owner-from-mr-participants into master

What does this MR do?

Code owner was included as part of approvers. However this would cause notifications and todos, which can spam code owners.

Prevent notifications/todos when new code owners becomes approvers.

  • RefreshService: this detects new code owners and sets them as approvers, now its notification creation is removed.
  • UpdateService: it finds difference in approver and sends out notifications to new approvers. Now code owners are excluded from this.
  • TodoService: Exclude code owners for todo creation3
  • MergeRequest#participant_approvers: Now exclude code owners

What are the relevant issue numbers?

Closes #8396 (closed)

Does this MR meet the acceptance criteria?

Edited by Mark Chao

Merge request reports