Ensure issuable state changes only fire webhooks once
requested to merge markglenfletcher/gitlab-ce:25339-2-webhooks-fired-for-issue-closed-and-reopened into master
What does this MR do?
Ensure issuable state changes only fire webhooks once
Are there points in the code the reviewer needs to double check?
Check to see if the tests could be refactored, possibly into a shared context as they are very similar?
Why was this MR needed?
Webhooks were fired twice when issuables were reopened or closed. Once for the status change and once for the update
operation
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
-
Changelog entry added - Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
Closes #25339 (closed)