-
Notifications
You must be signed in to change notification settings - Fork 235
Added create/delete users from teams, list repos of a team and create teams #104
Conversation
First attempt.
User can now add, delete users from a team and list repos of a team.
You can now create teams. Yay.
You can now create teams. yay.
rmUser, newTeam, repos, createTeam
I was just messing around, forgot that I had committed it.
@@ -31,6 +31,28 @@ class Team | |||
@client.get "/teams/#{@id}/members/#{user}", (err, s, b, h) -> | |||
return cb(err) if err | |||
cb null, s is 204, h | |||
|
|||
# Add a user to a team (must have admin permissions to do so) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra indentation in here.
Thanks for the pull request. 😄 Could you check my comments on the diff and fix them? Also please update README accordingly. Thanks |
Removed two lines at the bottom
Removed a line here or there, and renamed newUser to addUser, and rmUser to removeUser.
OK, I think that's everything. |
Added create/delete users from teams, list repos of a team and create teams
Very sorry, seems that I have given you the wrong version, the correct one must have been lost somewhere... For now I can't get to my repo because there is some 'maintenance going on', but I will get the correct version to you ASAP. Nothing major, but currently some of my implementations don't work. Sorry about this. |
hmm.. I checked the code. Didn't find any mistakes. Haven't tested it though. Could you fix them and send me PR again? Thanks |
I also compiled it and changed the documentation. Please do point out if I made a mistake somewhere 😄