Skip to content

Commit e2a0161

Browse files
authored
Merge pull request #2127 from objectcomputing/bugfix-2126/allow-feedback-reminders
Remove check for admin on email notification
2 parents ac6be6c + 4fc051f commit e2a0161

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

server/src/main/java/com/objectcomputing/checkins/notifications/email/MailJetNotificationController.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,12 @@ public MailJetNotificationController(CurrentUserServices currentUserServices,
3939

4040
@Post()
4141
public Mono<? extends HttpResponse<?>> sendEmailReceivesStatus(String subject, String content, String... recipients) {
42-
if (!currentUserServices.isAdmin()) {
43-
throw new PermissionException("You are not authorized to do this operation");
44-
}
45-
4642
MemberProfile currentUser = currentUserServices.getCurrentUser();
4743
String fromName = currentUser.getFirstName() + " " + currentUser.getLastName();
4844
return Mono.fromCallable(() -> emailSender.sendEmailReceivesStatus(fromName, currentUser.getWorkEmail(), subject, content, recipients))
4945
.publishOn(Schedulers.fromExecutor(eventLoopGroup))
5046
.map(success -> (HttpResponse<?>) HttpResponse.ok())
5147
.subscribeOn(Schedulers.fromExecutor(ioExecutorService));
52-
53-
5448
}
5549

5650
}

0 commit comments

Comments
 (0)