Skip to content

Exclude code owner from merge request participants to avoid spam

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

Loading