archive-gh-me/gitbeaker
1
0
Fork 0
mirror of https://github.com/jdalrymple/gitbeaker.git synced 2026-01-18 15:55:30 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #230 from maximilian-krauss/bugfix/add-slash-to-merge-requests

Browse Source 
fix: Adds slash after merge request id for editApprovals and editApprovers
This commit is contained in:
jdalrymple 2018-10-30 13:01:56 -04:00 committed by GitHub
commit 69335fdd84
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/services/MergeRequests.ts
View File

@ -86,14 +86,14 @@ class MergeRequests extends BaseService {
editApprovals(projectId, { mergerequestId, ...options }) {
const pId = encodeURIComponent(projectId);
const mergeRequest = mergerequestId ? `merge_requests/${encodeURIComponent(mergerequestId)}` : '';
const mergeRequest = mergerequestId ? `merge_requests/${encodeURIComponent(mergerequestId)}/` : '';
return RequestHelper.post(this, `projects/${pId}/${mergeRequest}approvals`, options);
}
editApprovers(projectId, { mergerequestId, ...options }) {
const pId = encodeURIComponent(projectId);
const mergeRequest = mergerequestId ? `merge_requests/${encodeURIComponent(mergerequestId)}` : '';
const mergeRequest = mergerequestId ? `merge_requests/${encodeURIComponent(mergerequestId)}/` : '';
return RequestHelper.put(this, `projects/${pId}/${mergeRequest}approvers`, options);
}