Skip to content

Commit 13266af

Browse files
authored
Merge pull request #113 from Team-Shaka/feat/104
🐛 푸시 알림 동의 여부 확인 대상 변경
2 parents b3c08fb + 9801bfa commit 13266af

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

src/main/java/treehouse/server/api/notification/business/NotificationService.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,10 @@ public void createNotification(User user, Long treehouseId, NotificationRequestD
4848
Member sender = memberQueryAdapter.findByUserAndTreehouse(user, treeHouse);
4949
User receiver = userQueryAdapter.findByIdOptional(request.getReceiverId()).orElse(null);
5050
Notification notification = NotificationMapper.toNotification(sender, receiver, request, reactionName);
51-
if(user.isPushAgree() && receiver != null) {
52-
fcmService.sendFcmMessage(receiver, notification.getTitle(), notification.getBody());
51+
52+
if(receiver.getUser().isPushAgree()) {
53+
fcmService.sendFcmMessage(receiver.getUser(), notification.getTitle(), notification.getBody());
54+
5355
}
5456
notificationCommandAdapter.createNotification(notification);
5557
}

src/main/java/treehouse/server/global/fcm/service/FcmService.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,6 @@ public UserResponseDTO.saveFcmToken saveFcmToken(User user, FCMDto.saveFcmTokenD
6666

6767

6868
public void sendFcmMessage(User receiver, String title, String body) {
69-
if (receiver.isPushAgree() == false) {
70-
return;
71-
}
7269

7370
List<FcmToken> fcmTokenList = fcmTokenRepository.findAllByUser(receiver);
7471
if (fcmTokenList.isEmpty()) {

0 commit comments

Comments
 (0)