Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support the check-user-team-membership API call #75

Merged
merged 1 commit into from
Mar 28, 2014

Conversation

rtyley
Copy link
Contributor

@rtyley rtyley commented Mar 12, 2014

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #136 SUCCESS
This pull request looks good
(what's this?)

rtyley added a commit to guardian/gu-who that referenced this pull request Mar 12, 2014
This rewrite does better at avoiding the awful race-condition where we
might add someone to the 'all' team immediately after they've been
removed from the Org. We check org-membership /after/ we've established
they're not in the 'all' team, and then add the user immediately, only if
they /are/ a member of the Org.

Note that membership of the 'all' team implies membership of the Org, so
no need to do the `user.isMemberOf(org)` call if the user is already a
member of the team.

hub4j/github-api#75
kohsuke added a commit that referenced this pull request Mar 28, 2014
Support the check-user-team-membership API call
@kohsuke kohsuke merged commit 5278ae3 into hub4j:master Mar 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants