Skip to content

Provide the ability to disable `all` mentions

Problem

  • @all is incredibly problematic and noisy.
  • It is a common occurrence for individuals to accidentally trigger this without meaning to or understanding the implications.
  • At the end of the day, pinging thousands of people in an issue or MR within a given project does not add a lot of value or utility.

Description

Currently, one can use @ all (space added to be safe) to contact all people in the project (and the group according to the auto-complete description), which is not quite desired in case of very large projects (we have other means for that).

Also, there's no way to contact all the people that were involved in a given merge request: watchers, those who replied in the thread, submitter and assignee. Sometimes people mistake the @ all for this purpose.

Finally, it would be useful to be able to use role names: @maintainers, @owners, @developers or @reporters.

Proposal

  • MVC 1: Put @all behind a FF so self-managed instances can disable @all notifications Task #414775 (closed)
  • MVC 2: Remove @all from the autocompletion list should be straightforward (weight 1.) #415523 (closed).

Availability and Testing

Regression testing, please make sure e2e:package-and-test job is passing.

Ensure updated test coverage in unit/integration tests.

Edited by Gabe Weaver