Skip to content

Commit 26064e5

Browse files
lunnyzeripath
andcommitted
Apply suggestions from code review
Co-authored-by: zeripath <[email protected]>
1 parent c732424 commit 26064e5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

modules/notification/mail/mail.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ func (m *mailNotifier) NotifyIssueChangeAssignee(doer *models.User, issue *model
105105
if !removed && doer.ID != assignee.ID && assignee.EmailNotifications() == models.EmailNotificationsEnabled {
106106
ct := fmt.Sprintf("Assigned #%d.", issue.Index)
107107
if err := mailer.SendIssueAssignedMail(issue, doer, ct, comment, []*models.User{assignee}); err != nil {
108-
log.Error("SendIssueAssignedMail faile: %v", err)
108+
log.Error("Error in SendIssueAssignedMail for issue[%d] to assignee[%d]: %v", issue.ID, assignee.ID, err)
109109
}
110110
}
111111
}
@@ -114,7 +114,7 @@ func (m *mailNotifier) NotifyPullReviewRequest(doer *models.User, issue *models.
114114
if isRequest && doer.ID != reviewer.ID && reviewer.EmailNotifications() == models.EmailNotificationsEnabled {
115115
ct := fmt.Sprintf("Requested to review %s.", issue.HTMLURL())
116116
if err := mailer.SendIssueAssignedMail(issue, doer, ct, comment, []*models.User{reviewer}); err != nil {
117-
log.Error("SendIssueAssignedMail faile: %v", err)
117+
log.Error("Error in SendIssueAssignedMail for issue[%d] to reviewer[%d]: %v", issue.ID, reviewer.ID, err)
118118
}
119119
}
120120
}

0 commit comments

Comments
 (0)