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

All examples at rest/api/vsts/git/pull%20requests/get%20pull%20requests look the same #343

Closed
kiewic opened this issue May 3, 2018 · 3 comments
Closed

All examples at rest/api/vsts/git/pull%20requests/get%20pull%20requests look the same #343

kiewic opened this issue May 3, 2018 · 3 comments
Assignees
@wnjenkin
Labels
content Issue with content on docs.microsoft.com transferred Issue was transferred somewhere else.

Comments

@kiewic
Copy link

kiewic commented May 3, 2018

The three examples in the VSTS Pull Requests REST API look the same

The three examples look like this:

image

Issue Details

I think the Sample Requests should be different to accomplish the different examples:

  1. Just completed pull requests
  2. Pull requests by repository
  3. Targeting a specific branch

Though I do not know if the query parameters are supposed to be passed in the query string or as a request payload.

Thanks!
@dend
Copy link
Contributor

dend commented May 3, 2018

@erickson-doug @hellosnow can you please take a look?

@dend dend added triaged assigned-to-pm A docs.microsoft.com PM is on the case! labels May 3, 2018
@dend dend assigned wnjenkin and unassigned erickson-doug Jul 4, 2018
@dend
Copy link
Contributor

dend commented Jul 4, 2018

@wnjenkin can you please take a look at this item?

@Powerhelmsman Powerhelmsman mentioned this issue Jul 9, 2018
Closed
@Powerhelmsman Powerhelmsman added the transferred Issue was transferred somewhere else. label Jul 9, 2018
@Powerhelmsman
Copy link
Collaborator

@kiewic Your issue is transferred to vsts-rest-api-specs repo. They will help you there.

@Powerhelmsman Powerhelmsman added content Issue with content on docs.microsoft.com and removed assigned-to-pm A docs.microsoft.com PM is on the case! labels Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wnjenkin wnjenkin

Labels
content Issue with content on docs.microsoft.com transferred Issue was transferred somewhere else.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kiewic @dend @erickson-doug @wnjenkin @Powerhelmsman