Skip to content

Commit f323120

Browse files
committed
fix bug
1 parent 8f5873d commit f323120

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

routers/api/v1/repo/pull_review.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -759,7 +759,7 @@ func apiReviewRequest(ctx *context.APIContext, opts api.PullReviewRequestOptions
759759
}
760760

761761
// DismissPullReview dismiss a review for a pull request
762-
func DismissPullReview(ctx *context.APIContext, opts api.DismissPullReviewOptions) {
762+
func DismissPullReview(ctx *context.APIContext) {
763763
// swagger:operation POST /repos/{owner}/{repo}/pulls/{index}/reviews/{id}/dismissals repository repoDismissPullReview
764764
// ---
765765
// summary: Dismiss a review for a pull request
@@ -800,6 +800,7 @@ func DismissPullReview(ctx *context.APIContext, opts api.DismissPullReviewOption
800800
// "$ref": "#/responses/forbidden"
801801
// "422":
802802
// "$ref": "#/responses/validationError"
803+
opts := web.GetForm(ctx).(*api.DismissPullReviewOptions)
803804
disMissReview(ctx, opts.Message, true)
804805
}
805806

routers/repo/pull_review.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,8 @@ func SubmitReview(ctx *context.Context) {
225225
}
226226

227227
// DismissReview dismissing stale review by repo admin
228-
func DismissReview(ctx *context.Context, form auth.DismissReviewForm) {
228+
func DismissReview(ctx *context.Context) {
229+
form := web.GetForm(ctx).(*auth.DismissReviewForm)
229230
comm, err := pull_service.DismissReview(form.ReviewID, form.Message, ctx.User, true)
230231
if err != nil {
231232
ctx.ServerError("pull_service.DismissReview", err)

0 commit comments

Comments
 (0)