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

Organization Membership Preview API Changes - existing endpoints #1514

Closed
ryangribble opened this issue Dec 12, 2016 · 4 comments
Closed

Organization Membership Preview API Changes - existing endpoints #1514

ryangribble opened this issue Dec 12, 2016 · 4 comments
Labels
Status: Stale Used by stalebot to clean house Status: Up for grabs Issues that are ready to be worked on by anyone

Comments

@ryangribble
Copy link
Contributor

As per this blog post several changes are in preview for the Organization Membership API

This issue is to cover changes to existing endpoints

  • The default_repository_permission and members_can_create_repositories settings for an organization can now be retrieved and set through the Get an organization and Edit an organization endpoints.

  • The Create team endpoint now offers the maintainers parameter which will allow you to add team maintainers upon creation.

  • You can now use the affiliation parameter with the List collaborators endpoint to filter a repository's collaborators by their affiliation type.

NOTE: A new preview accepts header needs to be specified application/vnd.github.korra-preview+json in AcceptHeaders.cs

@hnrkndrssn
Copy link
Contributor

And while I'm in the area, I'd like to put my face on this one too

@hnrkndrssn
Copy link
Contributor

hnrkndrssn commented Mar 9, 2019

You can now use the affiliation parameter with the List collaborators endpoint to filter a repository's collaborators by their affiliation type.

There is already a CollaboratorRequest class that is used when creating/updating a collaborator to set their permissions and I don't think I can re-use this class for the affiliation parameter (looking at the code it'd add all the properties of the class to the query string)?

Should I add a new class ListCollaboratorRequest(?) with the Affiliation property and use that for the List collaborators endpoint?

The Create team endpoint now offers the maintainers parameter which will allow you to add team maintainers upon creation.

Looks like this has already been implemented https://github.com/octokit/octokit.net/blob/master/Octokit/Models/Request/NewTeam.cs#L43

@ryangribble
Copy link
Contributor Author

Yeah it looks like the naming conventions have been a bit out of whack with this one. "normally" we would have XxxRequest on a GetAll endpoint, and NewXxx/UpdateXxx on any create/update endpoints... but in this case the CollaboratorRequest name has aleady been used 😬

So yeah I would say 👍 to this

Should I add a new class ListCollaboratorRequest(?) with the Affiliation property and use that for the List collaborators endpoint?

@github-actions
Copy link

github-actions bot commented Jul 3, 2022

👋 Hey Friends, this issue has been automatically marked as stale because it has no recent activity. It will be closed if no further activity occurs. Please add the pinned label if you feel that this issue needs to remain open/active. Thank you for your contributions and help in keeping things tidy!

@github-actions github-actions bot added the Stale label Jul 3, 2022
@nickfloyd nickfloyd added Status: Stale Used by stalebot to clean house Status: Up for grabs Issues that are ready to be worked on by anyone and removed Stale labels Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Stale Used by stalebot to clean house Status: Up for grabs Issues that are ready to be worked on by anyone
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants