Skip to content

Commit fa32742

Browse files
author
Ilya Radchenko
committed
Actually fix collaborators router
1 parent ed75a90 commit fa32742

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

lib/app.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ exports.init = function (config) {
197197
app.use(providerRouter);
198198

199199
// collaborators
200-
app.use('/:org/:repo/collaborators', collaboratorsRouter);
200+
app.use(collaboratorsRouter);
201201

202202
// branches
203203
app.use('/:org/:repo/branches/',

lib/routes/collaborators/index.js

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,12 @@ var api = require('./api');
1010
var router = new express.Router();
1111
var User = models.User;
1212

13-
router.route('/').all(
13+
router.route('/:org/:repo/collaborators/')
14+
.all(
1415
auth.requireUserOr401,
1516
middleware.project,
1617
auth.requireProjectAdmin
17-
);
18+
)
1819

1920
/**
2021
* @api {get} /:org/:repo/collaborators Get Collaborators
@@ -27,7 +28,7 @@ router.route('/').all(
2728
* @apiExample {curl} CURL Example:
2829
* curl -X GET http://localhost/api/strider-cd/strider/collaborators
2930
*/
30-
router.route('/').get(function getCollab(req, res) {
31+
.get(function getCollab(req, res) {
3132
var project = `${req.params.org}/${req.params.repo}`;
3233

3334
User.collaborators(project, 0, function (err, users) {
@@ -48,7 +49,7 @@ router.route('/').get(function getCollab(req, res) {
4849
}
4950
res.send(results);
5051
});
51-
});
52+
})
5253

5354
/**
5455
* @api {post} /:org/:repo/collaborators Add Collaborator
@@ -68,7 +69,7 @@ router.route('/').get(function getCollab(req, res) {
6869
* @apiParam (RequestBody) {Number} access=0 Access level to grant to the
6970
* new collaborator. This can be `0`, for read only access, or `2` for admin access.
7071
*/
71-
router.route('/').post(middleware.requireBody(['email']), function addCollab(req, res) {
72+
.post(middleware.requireBody(['email']), function addCollab(req, res) {
7273
var project = `${req.params.org}/${req.params.repo}`;
7374
var accessLevel = req.body.access || 0;
7475
var email = req.body.email;
@@ -87,7 +88,7 @@ router.route('/').post(middleware.requireBody(['email']), function addCollab(req
8788
message: `An invitation email has already been sent to ${email}. They will become a collaborator when they create an account.`
8889
});
8990
});
90-
});
91+
})
9192

9293
/**
9394
* @api {delete} /:org/:repo/collaborators Delete Collaborator
@@ -103,7 +104,7 @@ router.route('/').post(middleware.requireBody(['email']), function addCollab(req
103104
*
104105
* @apiParam (RequestBody) {String} email Email address to remove from the repo/project.
105106
*/
106-
router.route('/').delete(middleware.requireBody(['email']), function delCollab(req, res) {
107+
.delete(middleware.requireBody(['email']), function delCollab(req, res) {
107108
var project = `${req.params.org}/${req.params.repo}`;
108109
var email = req.body.email;
109110

0 commit comments

Comments
 (0)