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

Added .user_issues method to Issues #215

Merged
merged 2 commits into from Feb 8, 2013

Conversation

Projects
None yet
2 participants
@andrew
Contributor

andrew commented Feb 8, 2013

Which lists all issues across owned and member repositories for the authenticated user.

I made this a separate method to .list_issues as that already has two different api methods inside it and the docs are incorrect for when repo = nil

Added .user_issues method to Issues
Which lists all issues across owned and member repositories for the
authenticated user.
@pengwynn

This comment has been minimized.

Member

pengwynn commented Feb 8, 2013

Thanks for the patch. I believe the docs are correct. When repo = nil, it calls /issues which includes all visibile issues, not just for owned repos (basically includes teams, too).

Want to take a stab at the orgs version?

@andrew

This comment has been minimized.

Contributor

andrew commented Feb 8, 2013

Yep, I'll also go through and update the out of date docs since this pull was merged: #59

@andrew

This comment has been minimized.

Contributor

andrew commented Feb 8, 2013

@pengwynn where would be the best place to get some json for the org_issues.json fixture, I can't use work one as it has private info in it, or is it ok to reuse user_issues.json as the response will be in the same format?

@pengwynn

This comment has been minimized.

Member

pengwynn commented Feb 8, 2013

You can reuse/rename any of the existing issue arrays. It's the same content, just different API method.

Added .org_issues method
Which lists all issues for a given organization for the authenticated
user
@andrew

This comment has been minimized.

Contributor

andrew commented Feb 8, 2013

@pengwynn .org_issues added, I'll send in the doc changes as a separate pull request tomorrow, cheers!

pengwynn added a commit that referenced this pull request Feb 8, 2013

Merge pull request #215 from andrew/user-issues
Added .user_issues method to Issues

@pengwynn pengwynn merged commit 3d0b56c into octokit:master Feb 8, 2013

1 check was pending

default The Travis build is in progress
Details
@pengwynn

This comment has been minimized.

Member

pengwynn commented Feb 8, 2013

🍰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment