You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
forgejo/services
James Lakin 6fc129fe62
Fix repository create/delete event webhooks (#13008)
This small PR changes the webhook trigger behaviour to be more in line with what's expected. (When 'repository' events are enabled, of course)

In other words:

For system-wide or default webhooks, repository events will now trigger said webhook. Previously it had to be under an organization for create events to be visible - a tad unexpected!
Deleting a repository will now fire its own defined webhooks, not just organisational and system ones.
In order to enable the latter the webhook has to now be triggered before the actual repo undergoes deletion. I'm willing to tweak this to try and 'grab' the webhook model beforehand and trigger the webhook notifier directly afterwards, but this may make the code more complex for little benefit.

Closes #11766, #9180.
4 years ago
..
comments Refactor pull request review (#8954) 5 years ago
externalaccount Allow externalID to be UUID (#8551) 5 years ago
gitdiff Fix yet another bug with diff file names (#12771) 4 years ago
issue Change/remove a branch of an open issue (#9080) 4 years ago
mailer Fix "only mail on mention" bug (#12775) 4 years ago
mirror Mirror: Update DB on Address-Update too (#12964) 4 years ago
pull Add configurable Trust Models (#11712) 4 years ago
release Add a storage layer for attachments (#11387) 4 years ago
repository Fix repository create/delete event webhooks (#13008) 4 years ago
wiki Add configurable Trust Models (#11712) 4 years ago