diff options
Diffstat (limited to 'modules/notification/base')
| -rw-r--r-- | modules/notification/base/notifier.go | 11 | ||||
| -rw-r--r-- | modules/notification/base/null.go | 12 |
2 files changed, 14 insertions, 9 deletions
diff --git a/modules/notification/base/notifier.go b/modules/notification/base/notifier.go index 7b8eb7b1f1..b01026dfc5 100644 --- a/modules/notification/base/notifier.go +++ b/modules/notification/base/notifier.go @@ -20,7 +20,7 @@ type Notifier interface { NotifyRenameRepository(doer *models.User, repo *models.Repository, oldRepoName string) NotifyTransferRepository(doer *models.User, repo *models.Repository, oldOwnerName string) - NotifyNewIssue(*models.Issue) + NotifyNewIssue(issue *models.Issue, mentions []*models.User) NotifyIssueChangeStatus(*models.User, *models.Issue, *models.Comment, bool) NotifyIssueChangeMilestone(doer *models.User, issue *models.Issue, oldMilestoneID int64) NotifyIssueChangeAssignee(doer *models.User, issue *models.Issue, assignee *models.User, removed bool, comment *models.Comment) @@ -32,15 +32,16 @@ type Notifier interface { NotifyIssueChangeLabels(doer *models.User, issue *models.Issue, addedLabels []*models.Label, removedLabels []*models.Label) - NotifyNewPullRequest(*models.PullRequest) + NotifyNewPullRequest(pr *models.PullRequest, mentions []*models.User) NotifyMergePullRequest(*models.PullRequest, *models.User) NotifyPullRequestSynchronized(doer *models.User, pr *models.PullRequest) - NotifyPullRequestReview(*models.PullRequest, *models.Review, *models.Comment) + NotifyPullRequestReview(pr *models.PullRequest, review *models.Review, comment *models.Comment, mentions []*models.User) + NotifyPullRequestCodeComment(pr *models.PullRequest, comment *models.Comment, mentions []*models.User) NotifyPullRequestChangeTargetBranch(doer *models.User, pr *models.PullRequest, oldBranch string) NotifyPullRequestPushCommits(doer *models.User, pr *models.PullRequest, comment *models.Comment) - NotifyCreateIssueComment(*models.User, *models.Repository, - *models.Issue, *models.Comment) + NotifyCreateIssueComment(doer *models.User, repo *models.Repository, + issue *models.Issue, comment *models.Comment, mentions []*models.User) NotifyUpdateComment(*models.User, *models.Comment, string) NotifyDeleteComment(*models.User, *models.Comment) diff --git a/modules/notification/base/null.go b/modules/notification/base/null.go index 7a9ad0b2ce..d80ba859f3 100644 --- a/modules/notification/base/null.go +++ b/modules/notification/base/null.go @@ -23,11 +23,11 @@ func (*NullNotifier) Run() { // NotifyCreateIssueComment places a place holder function func (*NullNotifier) NotifyCreateIssueComment(doer *models.User, repo *models.Repository, - issue *models.Issue, comment *models.Comment) { + issue *models.Issue, comment *models.Comment, mentions []*models.User) { } // NotifyNewIssue places a place holder function -func (*NullNotifier) NotifyNewIssue(issue *models.Issue) { +func (*NullNotifier) NotifyNewIssue(issue *models.Issue, mentions []*models.User) { } // NotifyIssueChangeStatus places a place holder function @@ -35,11 +35,15 @@ func (*NullNotifier) NotifyIssueChangeStatus(doer *models.User, issue *models.Is } // NotifyNewPullRequest places a place holder function -func (*NullNotifier) NotifyNewPullRequest(pr *models.PullRequest) { +func (*NullNotifier) NotifyNewPullRequest(pr *models.PullRequest, mentions []*models.User) { } // NotifyPullRequestReview places a place holder function -func (*NullNotifier) NotifyPullRequestReview(pr *models.PullRequest, r *models.Review, comment *models.Comment) { +func (*NullNotifier) NotifyPullRequestReview(pr *models.PullRequest, r *models.Review, comment *models.Comment, mentions []*models.User) { +} + +// NotifyPullRequestCodeComment places a place holder function +func (*NullNotifier) NotifyPullRequestCodeComment(pr *models.PullRequest, comment *models.Comment, mentions []*models.User) { } // NotifyMergePullRequest places a place holder function |