Skip to content

Bug #1088 - Return proper URL on merge_request webhook

What does this MR do?

Adds an accepted case "merge_request" to the Gitlab::UrlBuilder which in turn returns a valid Merge Request URL

Are there points in the code the reviewer needs to double check?

No, this should be fairly straightforward

Why was this MR needed?

When the URL return value was added to the merge_request web hook in the following commit, a bug was created:

https://gitlab.com/gitlab-org/gitlab-ce/commit/4ffdb83e713b1ffa9578c02d31cfd0d9afe56ca7

This simple addition makes it so that when a request for a URL of a merge request is passed, a valid URL is returned. Tested and it provides valid output as expected.

What are the relevant issue numbers / Feature requests?

https://gitlab.com/gitlab-org/gitlab-ce/issues/1088

Merge request reports