这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Aug 23, 2024. It is now read-only.

Added create/delete users from teams, list repos of a team and create teams #104

Merged
merged 11 commits into from
Apr 2, 2014

Conversation

64
Copy link
Contributor

@64 64 commented Apr 1, 2014

I also compiled it and changed the documentation. Please do point out if I made a mistake somewhere 😄

64 added 8 commits March 31, 2014 20:12
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)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra indentation in here.

@pksunkara
Copy link
Owner

Thanks for the pull request. 😄

Could you check my comments on the diff and fix them? Also please update README accordingly.

Thanks

64 added 3 commits April 2, 2014 12:51
Removed two lines at the bottom
Removed  a line here or there, and renamed newUser to addUser, and rmUser to removeUser.
@64
Copy link
Contributor Author

64 commented Apr 2, 2014

OK, I think that's everything.

pksunkara added a commit that referenced this pull request Apr 2, 2014
Added create/delete users from teams, list repos of a team and create teams
@pksunkara pksunkara merged commit 98e2fee into pksunkara:master Apr 2, 2014
@64
Copy link
Contributor Author

64 commented Apr 2, 2014

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.

@pksunkara
Copy link
Owner

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants