-
Notifications
You must be signed in to change notification settings - Fork 15
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
[Feature Reqest] Add body
params for API calls
#29
Comments
This would be a great addition. The change to include a github-rest-apis-for-insomnia/index.js Line 64 in 31387a1
However, I don't think |
#7 should be easier @IAmHughes, since |
This might be worth revisiting now that |
Here's an example {
"content": {
"application/json": {
"schema": {
"type": "object",
"properties": {
"repository_ids": {
"type": "array",
"description": "The `id`s of the repositories that the installation token can access. Providing repository `id`s restricts the access of an installation token to specific repositories. You can use the \"[List repositories](https://developer.github.com/v3/apps/installations/#list-repositories)\" endpoint to get the `id` of all repositories that an installation can access. For example, you can select specific repositories when creating an installation token to restrict the number of repositories that can be cloned using the token.",
"items": {
"type": "integer"
}
},
"permissions": {
"type": "object",
"description": "The permissions granted to the access token. The permissions object includes the permission names and their access type. For a complete list of permissions and allowable values, see \"[GitHub App permissions](https://developer.github.com/apps/building-github-apps/creating-github-apps-using-url-parameters/#github-app-permissions).\"",
"properties": {}
}
}
},
"example": {
"repository_ids": [
1296269
],
"permissions": {
"issues": "write",
"contents": "read"
}
}
}
}
} |
I'd love to find a way to populate the
body
of requests too for each endpoint in theroutes
directory. The way this would look would be by modifying the empty"body": {}
to something like this:Here would be an example of the full object for one of the calls.
I spent about 30 minutes doing this by hand and it was not a fun process. We may want to consider programmatically getting this information in there somehow (I'm sure some fancy sed/awk can help).
Then I realized you used the Octokit Routes so I think we would need to update something in
Index.js
to pull thebody
tag out instead of just populating it with an empty body as we do hereThe text was updated successfully, but these errors were encountered: