Skip to content

Remove some unused code from the VisibleApprovable concern

What does this MR do?

While working on #11214 (closed) I discovered that none of these helpers seems to be used. We should remove them if that's the case - I spent a lot of time figuring them out unnecessarily :)

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

As long as the test suite is green, I think we're OK.

The smallest part of #33329 (closed)

Edited by Nick Thomas

Merge request reports