Skip to content

Unify project/group hooks codebase

Problem to solve

We have separate controllers for project and group webhooks:

And separate API controllers:

But they share the same view, which leads to issues like:

When we add some functionality to project level hooks, but forget to add it to group level.

Proposed solution

Unify some parts of parameter lists and specs. This way it would be harder to forget about group level webhooks.

Some downside - it will be harder to contribute FOSS only feature since it will require also contributing to EE.

Edited by Vladimir Shushlin
OSZAR »