Уведомление об изменении ревьюверов
Some checks failed
continuous-integration/drone/push Build is failing
Some checks failed
continuous-integration/drone/push Build is failing
This commit is contained in:
parent
625c59f163
commit
0e1e7c6cad
@ -68,11 +68,7 @@ public class MergeRequestHandler {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (checkNotEmpty(mergeRequest.getReviewers())) {
|
sendGroupNotifyForReviewers(mergeRequest, projectName);
|
||||||
sendGroupNotifyForReviewers(mergeRequest, projectName);
|
|
||||||
} else {
|
|
||||||
sendGroupNotifyAboutNoReviewers(mergeRequest, projectName);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -116,6 +112,9 @@ public class MergeRequestHandler {
|
|||||||
if (newConflict) {
|
if (newConflict) {
|
||||||
sendGroupNotifyAboutConflict(newMergeRequest, project.getName());
|
sendGroupNotifyAboutConflict(newMergeRequest, project.getName());
|
||||||
}
|
}
|
||||||
|
if (reviewerChanged.isChanged()) {
|
||||||
|
sendGroupNotifyForReviewers(newMergeRequest, project.getName());
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -156,22 +155,26 @@ public class MergeRequestHandler {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void sendGroupNotifyForReviewers(MergeRequest mergeRequest, String projectName) {
|
private void sendGroupNotifyForReviewers(MergeRequest mergeRequest, String projectName) {
|
||||||
final Map<Long, String> reviewerTelegramUsernames = personService.getTelegramUsernamesByPersonIds(mergeRequest.getReviewers().stream().map(Person::getId).collect(Collectors.toSet()));
|
if (checkNotEmpty(mergeRequest.getReviewers())) {
|
||||||
if (!reviewerTelegramUsernames.isEmpty()) {
|
final Map<Long, String> reviewerTelegramUsernames = personService.getTelegramUsernamesByPersonIds(mergeRequest.getReviewers().stream().map(Person::getId).collect(Collectors.toSet()));
|
||||||
notifyService.send(
|
if (!reviewerTelegramUsernames.isEmpty()) {
|
||||||
NewMergeRequestGroupNotify.builder()
|
notifyService.send(
|
||||||
.mrId(mergeRequest.getId())
|
NewMergeRequestGroupNotify.builder()
|
||||||
.title(mergeRequest.getTitle())
|
.mrId(mergeRequest.getId())
|
||||||
.url(mergeRequest.getWebUrl())
|
.title(mergeRequest.getTitle())
|
||||||
.author(mergeRequest.getAuthor().getName())
|
.url(mergeRequest.getWebUrl())
|
||||||
.reviewerTelegramUsernames(reviewerTelegramUsernames)
|
.author(mergeRequest.getAuthor().getName())
|
||||||
.milestone(mergeRequest.getMilestone())
|
.reviewerTelegramUsernames(reviewerTelegramUsernames)
|
||||||
.projectName(projectName)
|
.milestone(mergeRequest.getMilestone())
|
||||||
.targetBranch(mergeRequest.getTargetBranch())
|
.projectName(projectName)
|
||||||
.sourceBranch(mergeRequest.getSourceBranch())
|
.targetBranch(mergeRequest.getTargetBranch())
|
||||||
.description(mergeRequest.getDescription())
|
.sourceBranch(mergeRequest.getSourceBranch())
|
||||||
.build()
|
.description(mergeRequest.getDescription())
|
||||||
);
|
.build()
|
||||||
|
);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
sendGroupNotifyAboutNoReviewers(mergeRequest, projectName);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user