-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 preview Organization Project Permissions API #884
Comments
hello @gmlewis I would like to contribute to this issue. Thanks |
Thank you, @ganeshmannamal! I've sent you an invite. |
thanks @gmlewis accepted |
@gmlewis I see some PRs referenced here, has this been implemented? If not then I would like to take this up. |
Thanks, @palash25. I'm assigning this issue to you. If, during your investigation of this PR, you find there is nothing left to do, please report back with your findings. |
@gmlewis need a little help, I am confused about in which files to write the code for this issue. For the I have the same confusion with Organization projects on teams endpoints would the code go int |
I was thinking about putting them in projects.go and teams.go respectively. Would that be correct? |
Hi @palash25, sure. But first, can you please elaborate on which exact endpoints need to be written? We attempt to keep the code in this package organized as closely as possible to the GitHub Developer API documentation. So, for example, you can see the endpoints in
Then, depending upon the endpoints that still need to be written, we should be able to decide more easily where to put them. Does that make sense? |
@gmlewis These are the endpoints that I was talking about
I didn't find these endpoints implemented in the codebase. |
OK, great. Thank you for the details. The next thing I do is check where in the GitHub Developer API (V3) documentation these endpoints are found. In this case, we have:
and So it makes perfect sense to me to implement these in $ ls projects*.go teams*.go
projects.go teams_discussion_comments_test.go teams.go teams_test.go
projects_test.go teams_discussions.go teams_members.go
teams_discussion_comments.go teams_discussions_test.go teams_members_test.go and it doesn't look like we have Therefore, I think you are exactly right... these two new endpoints should be implemented in I know you don't need all this excruciating detail... but I took the time to write all this out for future maintainers and contributors of this package once my tenure is up. 😄 Thank you, @palash25! |
GitHub Developer API announcement:
https://developer.github.com/changes/2018-03-28-organization-project-permissions/
This would be a great PR for any new contributor to this repo or a new Go developer.
All contributions are greatly appreciated!
Feel free to volunteer for any issue, and we can send you an invite to contribute to the repo (which you then accept after you enable two-factor authentication) and the issue can be assigned to you so that others don't attempt to duplicate the work.
Please check out our CONTRIBUTING.md guide to get started.
Thank you!
The text was updated successfully, but these errors were encountered: