feat: add 'doer' to 'pullRequest' conversion for better PR review history

Makonike · Makonike · commit 4156990c7d92 · 2024-09-25T00:37:11.000+08:00
diff --git a/services/webhook/notifier.go b/services/webhook/notifier.go
@@ -378,7 +378,7 @@ func (m *webhookNotifier) UpdateComment(ctx context.Context, doer *user_model.Us
 	var pullRequest *api.PullRequest
 	if c.Issue.IsPull {
 		eventType = webhook_module.HookEventPullRequestComment
-		pullRequest = convert.ToAPIPullRequest(ctx, c.Issue.PullRequest, nil)
+		pullRequest = convert.ToAPIPullRequest(ctx, c.Issue.PullRequest, doer)
 	} else {
 		eventType = webhook_module.HookEventIssueComment
 	}

Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ func (m webhookNotifier) UpdateComment(ctx context.Context, doer user_model.Us`
`378`	`378`	`var pullRequest *api.PullRequest`
`379`	`379`	`if c.Issue.IsPull {`
`380`	`380`	`eventType = webhook_module.HookEventPullRequestComment`
`381`		`- pullRequest = convert.ToAPIPullRequest(ctx, c.Issue.PullRequest, nil)`
	`381`	`+ pullRequest = convert.ToAPIPullRequest(ctx, c.Issue.PullRequest, doer)`
`382`	`382`	`} else {`
`383`	`383`	`eventType = webhook_module.HookEventIssueComment`
`384`	`384`	`}`