wangwei
/
aiforge
Not watched
1
0
Fork 0
Code
Releases 128 Wiki Activity
Issues 0 Pull Requests 0 Datasets Model Cloudbrain
Browse Source

Fix swagger errors (#4220) 

Fix all the resting errors to have a valid swagger file.

They are still some warnings but nothing blocking.

Doing so I found that some request still misses son parameters for some POST/PUT/PATCH request. This means the a client generated from the swagger file will not work completely. 

Fix #4088 by activating validation in drone
Should fix #4010.
tags/v1.5.0-dev
Antoine GIRARD Bo-Yi Wu 7 years ago
parent
70cb88f17a
commit
00ff25393c
15 changed files with 432 additions and 254 deletions
Unified View
Diff Options
Show Stats Download Patch File Download Diff File
  1. +1
    -0
      .drone.yml
  2. +273
    -203
      public/swagger.v1.json
  3. +6
    -3
      routers/api/v1/misc/markdown.go
  4. +49
    -0
      routers/api/v1/org/hook.go
  5. +3
    -3
      routers/api/v1/org/team.go
  6. +1
    -1
      routers/api/v1/repo/file.go
  7. +6
    -1
      routers/api/v1/repo/hook.go
  8. +2
    -2
      routers/api/v1/repo/issue_comment.go
  9. +4
    -4
      routers/api/v1/repo/issue_tracked_time.go
  10. +24
    -19
      routers/api/v1/repo/milestone.go
  11. +6
    -6
      routers/api/v1/repo/repo.go
  12. +42
    -10
      routers/api/v1/swagger/options.go
  13. +1
    -0
      routers/api/v1/swagger/repo.go
  14. +12
    -0
      routers/api/v1/user/app.go
  15. +2
    -2
      routers/api/v1/user/follower.go

+ 1
- 0
.drone.yml View File

@@ -75,6 +75,7 @@ pipeline:
- make lint - make lint
- make fmt-check - make fmt-check
- make swagger-check - make swagger-check
- make swagger-validate
- make misspell-check - make misspell-check
- make test-vendor - make test-vendor
- make build - make build


+ 273
- 203
public/swagger.v1.json View File

@@ -321,9 +321,13 @@
"operationId": "renderMarkdownRaw", "operationId": "renderMarkdownRaw",
"parameters": [ "parameters": [
{ {
"type": "string",
"description": "Request body to render",
"name": "body", "name": "body",
"in": "body"
"in": "body",
"required": true,
"schema": {
"type": "string"
}
} }
], ],
"responses": { "responses": {
@@ -448,6 +452,15 @@
], ],
"summary": "List an organization's webhooks", "summary": "List an organization's webhooks",
"operationId": "orgListHooks", "operationId": "orgListHooks",
"parameters": [
{
"type": "string",
"description": "name of the organization",
"name": "org",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"200": { "200": {
"$ref": "#/responses/HookList" "$ref": "#/responses/HookList"
@@ -468,6 +481,15 @@
], ],
"summary": "Create a hook", "summary": "Create a hook",
"operationId": "orgCreateHook", "operationId": "orgCreateHook",
"parameters": [
{
"type": "string",
"description": "name of the organization",
"name": "org",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"201": { "201": {
"$ref": "#/responses/Hook" "$ref": "#/responses/Hook"
@@ -485,6 +507,22 @@
], ],
"summary": "Get a hook", "summary": "Get a hook",
"operationId": "orgGetHook", "operationId": "orgGetHook",
"parameters": [
{
"type": "string",
"description": "name of the organization",
"name": "org",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the hook to get",
"name": "id",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"200": { "200": {
"$ref": "#/responses/Hook" "$ref": "#/responses/Hook"
@@ -500,6 +538,22 @@
], ],
"summary": "Delete a hook", "summary": "Delete a hook",
"operationId": "orgDeleteHook", "operationId": "orgDeleteHook",
"parameters": [
{
"type": "string",
"description": "name of the organization",
"name": "org",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the hook to delete",
"name": "id",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"204": { "204": {
"$ref": "#/responses/empty" "$ref": "#/responses/empty"
@@ -518,6 +572,22 @@
], ],
"summary": "Update a hook", "summary": "Update a hook",
"operationId": "orgEditHook", "operationId": "orgEditHook",
"parameters": [
{
"type": "string",
"description": "name of the organization",
"name": "org",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the hook to update",
"name": "id",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"200": { "200": {
"$ref": "#/responses/Hook" "$ref": "#/responses/Hook"
@@ -994,7 +1064,7 @@
} }
} }
}, },
"/repos/{owner}/{repo}/archive/{filepath}": {
"/repos/{owner}/{repo}/archive/{archive}": {
"get": { "get": {
"produces": [ "produces": [
"application/json" "application/json"
@@ -1530,6 +1600,47 @@
} }
} }
}, },
"delete": {
"produces": [
"application/json"
],
"tags": [
"repository"
],
"summary": "Delete a hook in a repository",
"operationId": "repoDeleteHook",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the hook to delete",
"name": "id",
"in": "path",
"required": true
}
],
"responses": {
"204": {
"$ref": "#/responses/empty"
},
"404": {
"$ref": "#/responses/notFound"
}
}
},
"patch": { "patch": {
"produces": [ "produces": [
"application/json" "application/json"
@@ -1554,6 +1665,13 @@
"in": "path", "in": "path",
"required": true "required": true
}, },
{
"type": "integer",
"description": "index of the hook",
"name": "id",
"in": "path",
"required": true
},
{ {
"name": "body", "name": "body",
"in": "body", "in": "body",
@@ -1825,6 +1943,100 @@
} }
} }
}, },
"/repos/{owner}/{repo}/issues/{id}/times": {
"get": {
"produces": [
"application/json"
],
"tags": [
"issue"
],
"summary": "List an issue's tracked times",
"operationId": "issueTrackedTimes",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "index of the issue",
"name": "id",
"in": "path",
"required": true
}
],
"responses": {
"200": {
"$ref": "#/responses/TrackedTimeList"
}
}
},
"post": {
"consumes": [
"application/json"
],
"produces": [
"application/json"
],
"tags": [
"issue"
],
"summary": "Add a tracked time to a issue",
"operationId": "issueAddTime",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "index of the issue to add tracked time to",
"name": "id",
"in": "path",
"required": true
},
{
"name": "body",
"in": "body",
"schema": {
"$ref": "#/definitions/AddTimeOption"
}
}
],
"responses": {
"200": {
"$ref": "#/responses/TrackedTime"
},
"400": {
"$ref": "#/responses/error"
},
"403": {
"$ref": "#/responses/error"
}
}
}
},
"/repos/{owner}/{repo}/issues/{index}": { "/repos/{owner}/{repo}/issues/{index}": {
"get": { "get": {
"produces": [ "produces": [
@@ -1941,7 +2153,7 @@
{ {
"type": "integer", "type": "integer",
"description": "index of the issue", "description": "index of the issue",
"name": "id",
"name": "index",
"in": "path", "in": "path",
"required": true "required": true
}, },
@@ -1988,7 +2200,7 @@
{ {
"type": "integer", "type": "integer",
"description": "index of the issue", "description": "index of the issue",
"name": "id",
"name": "index",
"in": "path", "in": "path",
"required": true "required": true
}, },
@@ -2332,100 +2544,6 @@
} }
} }
}, },
"/repos/{owner}/{repo}/issues/{index}/times": {
"get": {
"produces": [
"application/json"
],
"tags": [
"issue"
],
"summary": "List an issue's tracked times",
"operationId": "issueTrackedTimes",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "index of the issue",
"name": "repo",
"in": "path",
"required": true
}
],
"responses": {
"200": {
"$ref": "#/responses/TrackedTimeList"
}
}
},
"post": {
"consumes": [
"application/json"
],
"produces": [
"application/json"
],
"tags": [
"issue"
],
"summary": "Add a tracked time to a issue",
"operationId": "issueAddTime",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "index of the issue to add tracked time to",
"name": "id",
"in": "path",
"required": true
},
{
"name": "body",
"in": "body",
"schema": {
"$ref": "#/definitions/AddTimeOption"
}
}
],
"responses": {
"200": {
"$ref": "#/responses/TrackedTime"
},
"400": {
"$ref": "#/responses/error"
},
"403": {
"$ref": "#/responses/error"
}
}
}
},
"/repos/{owner}/{repo}/keys": { "/repos/{owner}/{repo}/keys": {
"get": { "get": {
"produces": [ "produces": [
@@ -2781,7 +2899,7 @@
"issue" "issue"
], ],
"summary": "Get all of a repository's milestones", "summary": "Get all of a repository's milestones",
"operationId": "issueGetMilestones",
"operationId": "issueGetMilestonesList",
"parameters": [ "parameters": [
{ {
"type": "string", "type": "string",
@@ -2796,13 +2914,6 @@
"name": "repo", "name": "repo",
"in": "path", "in": "path",
"required": true "required": true
},
{
"type": "integer",
"description": "id of the milestone to get",
"name": "id",
"in": "path",
"required": true
} }
], ],
"responses": { "responses": {
@@ -2863,6 +2974,29 @@
], ],
"summary": "Get a milestone", "summary": "Get a milestone",
"operationId": "issueGetMilestone", "operationId": "issueGetMilestone",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the milestone",
"name": "id",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"200": { "200": {
"$ref": "#/responses/Milestone" "$ref": "#/responses/Milestone"
@@ -2893,7 +3027,7 @@
{ {
"type": "integer", "type": "integer",
"description": "id of the milestone to delete", "description": "id of the milestone to delete",
"name": "body",
"name": "id",
"in": "path", "in": "path",
"required": true "required": true
} }
@@ -2931,6 +3065,13 @@
"in": "path", "in": "path",
"required": true "required": true
}, },
{
"type": "integer",
"description": "id of the milestone",
"name": "id",
"in": "path",
"required": true
},
{ {
"name": "body", "name": "body",
"in": "body", "in": "body",
@@ -3979,7 +4120,7 @@
} }
} }
}, },
"/repos/{owner}/{repo}/times/{tracker}": {
"/repos/{owner}/{repo}/times/{user}": {
"get": { "get": {
"produces": [ "produces": [
"application/json" "application/json"
@@ -4019,49 +4160,6 @@
} }
} }
}, },
"/repos/{user}/{repo}/hooks/{id}": {
"delete": {
"produces": [
"application/json"
],
"tags": [
"repository"
],
"summary": "Delete a hook in a repository",
"operationId": "repoDeleteHook",
"parameters": [
{
"type": "string",
"description": "owner of the repo",
"name": "owner",
"in": "path",
"required": true
},
{
"type": "string",
"description": "name of the repo",
"name": "repo",
"in": "path",
"required": true
},
{
"type": "integer",
"description": "id of the hook to delete",
"name": "id",
"in": "path",
"required": true
}
],
"responses": {
"204": {
"$ref": "#/responses/empty"
},
"404": {
"$ref": "#/responses/notFound"
}
}
}
},
"/repositories/{id}": { "/repositories/{id}": {
"get": { "get": {
"produces": [ "produces": [
@@ -4238,7 +4336,7 @@
"organization" "organization"
], ],
"summary": "Remove a team member", "summary": "Remove a team member",
"operationId": "orgAddTeamMember",
"operationId": "orgRemoveTeamMember",
"parameters": [ "parameters": [
{ {
"type": "integer", "type": "integer",
@@ -4297,7 +4395,7 @@
"organization" "organization"
], ],
"summary": "Add a repository to a team", "summary": "Add a repository to a team",
"operationId": "orgAddTeamMember",
"operationId": "orgAddTeamRepository",
"parameters": [ "parameters": [
{ {
"type": "integer", "type": "integer",
@@ -4336,7 +4434,7 @@
"organization" "organization"
], ],
"summary": "Remove a repository from a team", "summary": "Remove a repository from a team",
"operationId": "orgAddTeamMember",
"operationId": "orgRemoveTeamRepository",
"parameters": [ "parameters": [
{ {
"type": "integer", "type": "integer",
@@ -4379,10 +4477,10 @@
"operationId": "topicSearch", "operationId": "topicSearch",
"parameters": [ "parameters": [
{ {
"type": "integer",
"description": "id of the repo to get",
"name": "keyword",
"in": "path",
"type": "string",
"description": "keywords to search",
"name": "q",
"in": "query",
"required": true "required": true
} }
], ],
@@ -4509,7 +4607,7 @@
} }
} }
}, },
"/user/following/{followee}": {
"/user/following/{username}": {
"get": { "get": {
"tags": [ "tags": [
"user" "user"
@@ -4520,7 +4618,7 @@
{ {
"type": "string", "type": "string",
"description": "username of followed user", "description": "username of followed user",
"name": "followee",
"name": "username",
"in": "path", "in": "path",
"required": true "required": true
} }
@@ -4533,9 +4631,7 @@
"$ref": "#/responses/notFound" "$ref": "#/responses/notFound"
} }
} }
}
},
"/user/following/{username}": {
},
"put": { "put": {
"tags": [ "tags": [
"user" "user"
@@ -5301,6 +5397,15 @@
], ],
"summary": "List the authenticated user's access tokens", "summary": "List the authenticated user's access tokens",
"operationId": "userGetTokens", "operationId": "userGetTokens",
"parameters": [
{
"type": "string",
"description": "username of user",
"name": "username",
"in": "path",
"required": true
}
],
"responses": { "responses": {
"200": { "200": {
"$ref": "#/responses/AccessTokenList" "$ref": "#/responses/AccessTokenList"
@@ -5323,8 +5428,10 @@
{ {
"type": "string", "type": "string",
"x-go-name": "Name", "x-go-name": "Name",
"name": "name",
"in": "query"
"description": "username of user",
"name": "username",
"in": "path",
"required": true
} }
], ],
"responses": { "responses": {
@@ -7612,9 +7719,6 @@
"description": "SearchResults", "description": "SearchResults",
"schema": { "schema": {
"$ref": "#/definitions/SearchResults" "$ref": "#/definitions/SearchResults"
},
"headers": {
"body": {}
} }
}, },
"ServerVersion": { "ServerVersion": {
@@ -7721,40 +7825,6 @@
"description": "parameterBodies", "description": "parameterBodies",
"schema": { "schema": {
"$ref": "#/definitions/EditAttachmentOptions" "$ref": "#/definitions/EditAttachmentOptions"
},
"headers": {
"AddCollaboratorOption": {},
"AddTimeOption": {},
"CreateEmailOption": {},
"CreateForkOption": {},
"CreateHookOption": {},
"CreateIssueCommentOption": {},
"CreateIssueOption": {},
"CreateKeyOption": {},
"CreateLabelOption": {},
"CreateMilestoneOption": {},
"CreateOrgOption": {},
"CreatePullRequestOption": {},
"CreateReleaseOption": {},
"CreateRepoOption": {},
"CreateStatusOption": {},
"CreateTeamOption": {},
"CreateUserOption": {},
"DeleteEmailOption": {},
"EditAttachmentOptions": {},
"EditHookOption": {},
"EditIssueCommentOption": {},
"EditIssueOption": {},
"EditLabelOption": {},
"EditMilestoneOption": {},
"EditOrgOption": {},
"EditPullRequestOption": {},
"EditReleaseOption": {},
"EditTeamOption": {},
"EditUserOption": {},
"IssueLabelsOption": {},
"MarkdownOption": {},
"MigrateRepoForm": {}
} }
}, },
"redirect": { "redirect": {


+ 6
- 3
routers/api/v1/misc/markdown.go View File

@@ -62,9 +62,12 @@ func MarkdownRaw(ctx *context.APIContext) {
// --- // ---
// summary: Render raw markdown as HTML // summary: Render raw markdown as HTML
// parameters: // parameters:
// - name: body
// in: body
// type: string
// - name: body
// in: body
// description: Request body to render
// required: true
// schema:
// type: string
// consumes: // consumes:
// - text/plain // - text/plain
// produces: // produces:


+ 49
- 0
routers/api/v1/org/hook.go View File

@@ -20,6 +20,12 @@ func ListHooks(ctx *context.APIContext) {
// summary: List an organization's webhooks // summary: List an organization's webhooks
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: org
// in: path
// description: name of the organization
// type: string
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/HookList" // "$ref": "#/responses/HookList"
@@ -43,6 +49,17 @@ func GetHook(ctx *context.APIContext) {
// summary: Get a hook // summary: Get a hook
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: org
// in: path
// description: name of the organization
// type: string
// required: true
// - name: id
// in: path
// description: id of the hook to get
// type: integer
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/Hook" // "$ref": "#/responses/Hook"
@@ -64,9 +81,17 @@ func CreateHook(ctx *context.APIContext, form api.CreateHookOption) {
// - application/json // - application/json
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: org
// in: path
// description: name of the organization
// type: string
// required: true
// responses: // responses:
// "201": // "201":
// "$ref": "#/responses/Hook" // "$ref": "#/responses/Hook"

//TODO in body params
if !utils.CheckCreateHookOption(ctx, &form) { if !utils.CheckCreateHookOption(ctx, &form) {
return return
} }
@@ -82,9 +107,22 @@ func EditHook(ctx *context.APIContext, form api.EditHookOption) {
// - application/json // - application/json
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: org
// in: path
// description: name of the organization
// type: string
// required: true
// - name: id
// in: path
// description: id of the hook to update
// type: integer
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/Hook" // "$ref": "#/responses/Hook"

//TODO in body params
hookID := ctx.ParamsInt64(":id") hookID := ctx.ParamsInt64(":id")
utils.EditOrgHook(ctx, &form, hookID) utils.EditOrgHook(ctx, &form, hookID)
} }
@@ -96,6 +134,17 @@ func DeleteHook(ctx *context.APIContext) {
// summary: Delete a hook // summary: Delete a hook
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: org
// in: path
// description: name of the organization
// type: string
// required: true
// - name: id
// in: path
// description: id of the hook to delete
// type: integer
// required: true
// responses: // responses:
// "204": // "204":
// "$ref": "#/responses/empty" // "$ref": "#/responses/empty"


+ 3
- 3
routers/api/v1/org/team.go View File

@@ -227,7 +227,7 @@ func AddTeamMember(ctx *context.APIContext) {


// RemoveTeamMember api for remove one member from a team // RemoveTeamMember api for remove one member from a team
func RemoveTeamMember(ctx *context.APIContext) { func RemoveTeamMember(ctx *context.APIContext) {
// swagger:operation DELETE /teams/{id}/members/{username} organization orgAddTeamMember
// swagger:operation DELETE /teams/{id}/members/{username} organization orgRemoveTeamMember
// --- // ---
// summary: Remove a team member // summary: Remove a team member
// produces: // produces:
@@ -306,7 +306,7 @@ func getRepositoryByParams(ctx *context.APIContext) *models.Repository {


// AddTeamRepository api for adding a repository to a team // AddTeamRepository api for adding a repository to a team
func AddTeamRepository(ctx *context.APIContext) { func AddTeamRepository(ctx *context.APIContext) {
// swagger:operation PUT /teams/{id}/repos/{org}/{repo} organization orgAddTeamMember
// swagger:operation PUT /teams/{id}/repos/{org}/{repo} organization orgAddTeamRepository
// --- // ---
// summary: Add a repository to a team // summary: Add a repository to a team
// produces: // produces:
@@ -350,7 +350,7 @@ func AddTeamRepository(ctx *context.APIContext) {


// RemoveTeamRepository api for removing a repository from a team // RemoveTeamRepository api for removing a repository from a team
func RemoveTeamRepository(ctx *context.APIContext) { func RemoveTeamRepository(ctx *context.APIContext) {
// swagger:operation DELETE /teams/{id}/repos/{org}/{repo} organization orgAddTeamMember
// swagger:operation DELETE /teams/{id}/repos/{org}/{repo} organization orgRemoveTeamRepository
// --- // ---
// summary: Remove a repository from a team // summary: Remove a repository from a team
// description: This does not delete the repository, it only removes the // description: This does not delete the repository, it only removes the


+ 1
- 1
routers/api/v1/repo/file.go View File

@@ -64,7 +64,7 @@ func GetRawFile(ctx *context.APIContext) {


// GetArchive get archive of a repository // GetArchive get archive of a repository
func GetArchive(ctx *context.APIContext) { func GetArchive(ctx *context.APIContext) {
// swagger:operation GET /repos/{owner}/{repo}/archive/{filepath} repository repoGetArchive
// swagger:operation GET /repos/{owner}/{repo}/archive/{archive} repository repoGetArchive
// --- // ---
// summary: Get an archive of a repository // summary: Get an archive of a repository
// produces: // produces:


+ 6
- 1
routers/api/v1/repo/hook.go View File

@@ -189,6 +189,11 @@ func EditHook(ctx *context.APIContext, form api.EditHookOption) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: id
// in: path
// description: index of the hook
// type: integer
// required: true
// - name: body // - name: body
// in: body // in: body
// schema: // schema:
@@ -202,7 +207,7 @@ func EditHook(ctx *context.APIContext, form api.EditHookOption) {


// DeleteHook delete a hook of a repository // DeleteHook delete a hook of a repository
func DeleteHook(ctx *context.APIContext) { func DeleteHook(ctx *context.APIContext) {
// swagger:operation DELETE /repos/{user}/{repo}/hooks/{id} repository repoDeleteHook
// swagger:operation DELETE /repos/{owner}/{repo}/hooks/{id} repository repoDeleteHook
// --- // ---
// summary: Delete a hook in a repository // summary: Delete a hook in a repository
// produces: // produces:


+ 2
- 2
routers/api/v1/repo/issue_comment.go View File

@@ -31,7 +31,7 @@ func ListIssueComments(ctx *context.APIContext) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: id
// - name: index
// in: path // in: path
// description: index of the issue // description: index of the issue
// type: integer // type: integer
@@ -139,7 +139,7 @@ func CreateIssueComment(ctx *context.APIContext, form api.CreateIssueCommentOpti
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: id
// - name: index
// in: path // in: path
// description: index of the issue // description: index of the issue
// type: integer // type: integer


+ 4
- 4
routers/api/v1/repo/issue_tracked_time.go View File

@@ -21,7 +21,7 @@ func trackedTimesToAPIFormat(trackedTimes []*models.TrackedTime) []*api.TrackedT


// ListTrackedTimes list all the tracked times of an issue // ListTrackedTimes list all the tracked times of an issue
func ListTrackedTimes(ctx *context.APIContext) { func ListTrackedTimes(ctx *context.APIContext) {
// swagger:operation GET /repos/{owner}/{repo}/issues/{index}/times issue issueTrackedTimes
// swagger:operation GET /repos/{owner}/{repo}/issues/{id}/times issue issueTrackedTimes
// --- // ---
// summary: List an issue's tracked times // summary: List an issue's tracked times
// produces: // produces:
@@ -37,7 +37,7 @@ func ListTrackedTimes(ctx *context.APIContext) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: repo
// - name: id
// in: path // in: path
// description: index of the issue // description: index of the issue
// type: integer // type: integer
@@ -70,7 +70,7 @@ func ListTrackedTimes(ctx *context.APIContext) {


// AddTime adds time manual to the given issue // AddTime adds time manual to the given issue
func AddTime(ctx *context.APIContext, form api.AddTimeOption) { func AddTime(ctx *context.APIContext, form api.AddTimeOption) {
// swagger:operation Post /repos/{owner}/{repo}/issues/{index}/times issue issueAddTime
// swagger:operation Post /repos/{owner}/{repo}/issues/{id}/times issue issueAddTime
// --- // ---
// summary: Add a tracked time to a issue // summary: Add a tracked time to a issue
// consumes: // consumes:
@@ -132,7 +132,7 @@ func AddTime(ctx *context.APIContext, form api.AddTimeOption) {


// ListTrackedTimesByUser lists all tracked times of the user // ListTrackedTimesByUser lists all tracked times of the user
func ListTrackedTimesByUser(ctx *context.APIContext) { func ListTrackedTimesByUser(ctx *context.APIContext) {
// swagger:operation GET /repos/{owner}/{repo}/times/{tracker} user userTrackedTimes
// swagger:operation GET /repos/{owner}/{repo}/times/{user} user userTrackedTimes
// --- // ---
// summary: List a user's tracked times in a repo // summary: List a user's tracked times in a repo
// produces: // produces:


+ 24
- 19
routers/api/v1/repo/milestone.go View File

@@ -16,14 +16,25 @@ import (


// ListMilestones list all the milestones for a repository // ListMilestones list all the milestones for a repository
func ListMilestones(ctx *context.APIContext) { func ListMilestones(ctx *context.APIContext) {
// swagger:operation GET /repos/{owner}/{repo}/milestones/{id} issue issueGetMilestone
// swagger:operation GET /repos/{owner}/{repo}/milestones issue issueGetMilestonesList
// --- // ---
// summary: Get a milestone
// summary: Get all of a repository's milestones
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: owner
// in: path
// description: owner of the repo
// type: string
// required: true
// - name: repo
// in: path
// description: name of the repo
// type: string
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/Milestone"
// "$ref": "#/responses/MilestoneList"
milestones, err := models.GetMilestonesByRepoID(ctx.Repo.Repository.ID) milestones, err := models.GetMilestonesByRepoID(ctx.Repo.Repository.ID)
if err != nil { if err != nil {
ctx.Error(500, "GetMilestonesByRepoID", err) ctx.Error(500, "GetMilestonesByRepoID", err)
@@ -39,9 +50,9 @@ func ListMilestones(ctx *context.APIContext) {


// GetMilestone get a milestone for a repository // GetMilestone get a milestone for a repository
func GetMilestone(ctx *context.APIContext) { func GetMilestone(ctx *context.APIContext) {
// swagger:operation GET /repos/{owner}/{repo}/milestones issue issueGetMilestones
// swagger:operation GET /repos/{owner}/{repo}/milestones/{id} issue issueGetMilestone
// --- // ---
// summary: Get all of a repository's milestones
// summary: Get a milestone
// produces: // produces:
// - application/json // - application/json
// parameters: // parameters:
@@ -55,25 +66,14 @@ func GetMilestone(ctx *context.APIContext) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// parameters:
// - name: owner
// in: path
// description: owner of the repo
// type: string
// required: true
// - name: repo
// in: path
// description: name of the repo
// type: string
// required: true
// - name: id // - name: id
// in: path // in: path
// description: id of the milestone to get
// description: id of the milestone
// type: integer // type: integer
// required: true // required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/MilestoneList"
// "$ref": "#/responses/Milestone"
milestone, err := models.GetMilestoneByRepoID(ctx.Repo.Repository.ID, ctx.ParamsInt64(":id")) milestone, err := models.GetMilestoneByRepoID(ctx.Repo.Repository.ID, ctx.ParamsInt64(":id"))
if err != nil { if err != nil {
if models.IsErrMilestoneNotExist(err) { if models.IsErrMilestoneNotExist(err) {
@@ -152,6 +152,11 @@ func EditMilestone(ctx *context.APIContext, form api.EditMilestoneOption) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: id
// in: path
// description: id of the milestone
// type: integer
// required: true
// - name: body // - name: body
// in: body // in: body
// schema: // schema:
@@ -202,7 +207,7 @@ func DeleteMilestone(ctx *context.APIContext) {
// description: name of the repo // description: name of the repo
// type: string // type: string
// required: true // required: true
// - name: body
// - name: id
// in: path // in: path
// description: id of the milestone to delete // description: id of the milestone to delete
// type: integer // type: integer


+ 6
- 6
routers/api/v1/repo/repo.go View File

@@ -508,13 +508,13 @@ func TopicSearch(ctx *context.Context) {
// --- // ---
// summary: search topics via keyword // summary: search topics via keyword
// produces: // produces:
// - application/json
// - application/json
// parameters: // parameters:
// - name: keyword
// in: path
// description: id of the repo to get
// type: integer
// required: true
// - name: q
// in: query
// description: keywords to search
// required: true
// type: string
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/Repository" // "$ref": "#/responses/Repository"


+ 42
- 10
routers/api/v1/swagger/options.go View File

@@ -15,55 +15,87 @@ import (
// parameterBodies // parameterBodies
// swagger:response parameterBodies // swagger:response parameterBodies
type swaggerParameterBodies struct { type swaggerParameterBodies struct {
// in:body
AddCollaboratorOption api.AddCollaboratorOption AddCollaboratorOption api.AddCollaboratorOption


// in:body
CreateEmailOption api.CreateEmailOption CreateEmailOption api.CreateEmailOption
// in:body
DeleteEmailOption api.DeleteEmailOption DeleteEmailOption api.DeleteEmailOption


// in:body
CreateHookOption api.CreateHookOption CreateHookOption api.CreateHookOption
EditHookOption api.EditHookOption
// in:body
EditHookOption api.EditHookOption


// in:body
CreateIssueOption api.CreateIssueOption CreateIssueOption api.CreateIssueOption
EditIssueOption api.EditIssueOption
// in:body
EditIssueOption api.EditIssueOption


// in:body
CreateIssueCommentOption api.CreateIssueCommentOption CreateIssueCommentOption api.CreateIssueCommentOption
EditIssueCommentOption api.EditIssueCommentOption
// in:body
EditIssueCommentOption api.EditIssueCommentOption


// in:body
IssueLabelsOption api.IssueLabelsOption IssueLabelsOption api.IssueLabelsOption


// in:body
CreateKeyOption api.CreateKeyOption CreateKeyOption api.CreateKeyOption


// in:body
CreateLabelOption api.CreateLabelOption CreateLabelOption api.CreateLabelOption
EditLabelOption api.EditLabelOption
// in:body
EditLabelOption api.EditLabelOption


// in:body
MarkdownOption api.MarkdownOption MarkdownOption api.MarkdownOption


// in:body
CreateMilestoneOption api.CreateMilestoneOption CreateMilestoneOption api.CreateMilestoneOption
EditMilestoneOption api.EditMilestoneOption
// in:body
EditMilestoneOption api.EditMilestoneOption


// in:body
CreateOrgOption api.CreateOrgOption CreateOrgOption api.CreateOrgOption
EditOrgOption api.EditOrgOption
// in:body
EditOrgOption api.EditOrgOption


// in:body
CreatePullRequestOption api.CreatePullRequestOption CreatePullRequestOption api.CreatePullRequestOption
EditPullRequestOption api.EditPullRequestOption
// in:body
EditPullRequestOption api.EditPullRequestOption


// in:body
CreateReleaseOption api.CreateReleaseOption CreateReleaseOption api.CreateReleaseOption
EditReleaseOption api.EditReleaseOption
// in:body
EditReleaseOption api.EditReleaseOption


// in:body
CreateRepoOption api.CreateRepoOption CreateRepoOption api.CreateRepoOption
// in:body
CreateForkOption api.CreateForkOption CreateForkOption api.CreateForkOption


// in:body
CreateStatusOption api.CreateStatusOption CreateStatusOption api.CreateStatusOption


// in:body
CreateTeamOption api.CreateTeamOption CreateTeamOption api.CreateTeamOption
EditTeamOption api.EditTeamOption
// in:body
EditTeamOption api.EditTeamOption


// in:body
AddTimeOption api.AddTimeOption AddTimeOption api.AddTimeOption


// in:body
CreateUserOption api.CreateUserOption CreateUserOption api.CreateUserOption
EditUserOption api.EditUserOption
// in:body
EditUserOption api.EditUserOption


// in:body
MigrateRepoForm auth.MigrateRepoForm MigrateRepoForm auth.MigrateRepoForm


// in:body
EditAttachmentOptions api.EditAttachmentOptions EditAttachmentOptions api.EditAttachmentOptions
} }

+ 1
- 0
routers/api/v1/swagger/repo.go View File

@@ -102,6 +102,7 @@ type swaggerResponseWatchInfo struct {
// SearchResults // SearchResults
// swagger:response SearchResults // swagger:response SearchResults
type swaggerResponseSearchResults struct { type swaggerResponseSearchResults struct {
// in:body
Body api.SearchResults `json:"body"` Body api.SearchResults `json:"body"`
} }




+ 12
- 0
routers/api/v1/user/app.go View File

@@ -18,6 +18,12 @@ func ListAccessTokens(ctx *context.APIContext) {
// summary: List the authenticated user's access tokens // summary: List the authenticated user's access tokens
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: username
// in: path
// description: username of user
// type: string
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/AccessTokenList" // "$ref": "#/responses/AccessTokenList"
@@ -46,6 +52,12 @@ func CreateAccessToken(ctx *context.APIContext, form api.CreateAccessTokenOption
// - application/json // - application/json
// produces: // produces:
// - application/json // - application/json
// parameters:
// - name: username
// in: path
// description: username of user
// type: string
// required: true
// responses: // responses:
// "200": // "200":
// "$ref": "#/responses/AccessToken" // "$ref": "#/responses/AccessToken"


+ 2
- 2
routers/api/v1/user/follower.go View File

@@ -119,11 +119,11 @@ func checkUserFollowing(ctx *context.APIContext, u *models.User, followID int64)


// CheckMyFollowing whether the given user is followed by the authenticated user // CheckMyFollowing whether the given user is followed by the authenticated user
func CheckMyFollowing(ctx *context.APIContext) { func CheckMyFollowing(ctx *context.APIContext) {
// swagger:operation GET /user/following/{followee} user userCurrentCheckFollowing
// swagger:operation GET /user/following/{username} user userCurrentCheckFollowing
// --- // ---
// summary: Check whether a user is followed by the authenticated user // summary: Check whether a user is followed by the authenticated user
// parameters: // parameters:
// - name: followee
// - name: username
// in: path // in: path
// description: username of followed user // description: username of followed user
// type: string // type: string


Write Preview
Loading…
Cancel
Save