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.

Sign up for GitHub

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

Jump to bottom

Return owner team members on the Organization endpoint #1734

Merged
merged 1 commit into from
Dec 5, 2019
Merged

Return owner team members on the Organization endpoint #1734

merged 1 commit into from
Dec 5, 2019

Conversation

DavisRayM
Copy link
Contributor

@DavisRayM DavisRayM commented Dec 2, 2019 •
edited
Loading

Changes

  • Return members of the 'Owners' team on the /orgs endpoint under the users key.

@DavisRayM DavisRayM force-pushed the set-org-creator-owner-perm branch 6 times, most recently from 570cb88 to d744427 Compare December 2, 2019 12:04
@DavisRayM DavisRayM requested a review from ukanga December 2, 2019 12:16
@DavisRayM DavisRayM force-pushed the set-org-creator-owner-perm branch 5 times, most recently from 1bf60ed to 453a8c9 Compare December 2, 2019 14:20
@DavisRayM DavisRayM force-pushed the set-org-creator-owner-perm branch 6 times, most recently from d137fcf to 373e566 Compare December 3, 2019 11:11
@DavisRayM
Return org owner team under 'users' on /org
401019f 
- Update tests
- Add get_organization_owners function
- Add test_creator_in_users
@DavisRayM DavisRayM changed the title Set owner permission for Organization creator Return owner team members on the Organization endpoint Dec 5, 2019
@ukanga ukanga merged commit e4fe81c into master Dec 5, 2019
@ukanga ukanga deleted the set-org-creator-owner-perm branch December 5, 2019 12:38
@DavisRayM DavisRayM mentioned this pull request Dec 9, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ukanga ukanga ukanga approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DavisRayM @ukanga