#40,41,42,43,44,45,46,47 - 알림 관련 기능 구현했습니다. #61
Conversation
| FirebaseApp.initializeApp(options) | ||
| } | ||
| } catch (e: Exception) { | ||
| e.printStackTrace() |
| class NotificationServiceHelperTest @Autowired constructor( | ||
| private val notificationServiceHelper: NotificationServiceHelper, | ||
| private val databaseCleanup: DatabaseCleanup | ||
| ) { // TODO : Mocking 고수 기천짱 나를 도와줘.. |
There was a problem hiding this comment.
기천짱 요기 도와줘잉 verify 가 원하는대로 동작하지를 않아아아아
| eventPublisher.publishEvent( | ||
| ReadMessageByReceiverEvent( | ||
| sender, | ||
| loginUser, | ||
| readMessage.id, | ||
| message.isReceiverRead | ||
| ) | ||
| ) |
There was a problem hiding this comment.
수신자가 메시지를 처음 읽는 경우 알림을 보내줄 수 있도록 이벤트를 발생시키고 있어요
| ) // TODO : 나중에 메시지 예약 취소하는 경우에도 해당 이벤트 발생시켜주시면 좋을 것 같아요. | ||
| // eventPublisher.publishEvent(ReceivedMessageEvent(receiver, saveMessage.id)) // TODO : 생각해보니 이것은 실제로 발송될 때, 전송되어야 하니 나중에 스케줄링 추가하실 때 해당 이벤트 추가해주실 수 있을까요? |
There was a problem hiding this comment.
이건 생각해보니, 이미 적용이 되었었던가 싶네요. 이거 이미 적용되어있나 기천짱?
There was a problem hiding this comment.
노노 개발해야됩니다ㅜ 나중에 개발할때 제가 추가하거나 말씀드릴게요!
| import org.springframework.transaction.annotation.Transactional | ||
|
|
||
| @Service | ||
| class CreatedMessageNotificationService( |
There was a problem hiding this comment.
여기는 메시지 관련 알림을 생성하고, 푸시 알림을 보내는 서비스에요
| import org.springframework.stereotype.Component | ||
|
|
||
| @Component | ||
| class VoteNotificationEventListener( |
| requests.voteRequests | ||
| .stream() | ||
| .forEach { request -> addBallot(request, vote, user, voteTime) } | ||
| eventPublisher.publishEvent(RegisteredVoteEvent(user, vote)) |
There was a problem hiding this comment.
유저가 투표를 등록했을 때 이벤트에요
| voteTime: LocalDateTime | ||
| ) { | ||
| val receiver = VoteServiceHelper.findUser(userPort, request.userId) | ||
| eventPublisher.publishEvent(ReceivedVoteEvent(receiver)) |
There was a problem hiding this comment.
실제로 투표를 받은 이에게 알림을 보낼 수 있도록 해주는 이벤트에요
| val readAt: LocalDateTime, | ||
| val isEnabled: Boolean, | ||
| val title: String, | ||
| val body: String, |
There was a problem hiding this comment.
알림에 추가적인 사항이 생겼어요 이로 인해서 명시적으로 title과 body를 분리했어요
| } | ||
|
|
||
| private fun isNotNotificationTrigger(numberOfSender: Int) = | ||
| numberOfSender < 5 || (numberOfSender - 5) % 3 != 0 |
There was a problem hiding this comment.
초기 5명 이후 3명마다 알림 발생하는 로직이에요
| return ResponseEntity.ok(notifications) | ||
| } | ||
|
|
||
| @PatchMapping("{notificationId}") |
| } | ||
|
|
||
| @PatchMapping("{notificationId}") | ||
| fun readNotification(@PathVariable notificationId: Long): ResponseEntity<Void> { |
There was a problem hiding this comment.
Void말고 코틀린이니 Unit으로 해보는건 어때요 ??
편하신대로 해주시면 떙큐!
There was a problem hiding this comment.
헐 Void 했네 ㅠㅠㅠ 미안해용 고칠게요 Java 저리갓!
| ) // TODO : 나중에 메시지 예약 취소하는 경우에도 해당 이벤트 발생시켜주시면 좋을 것 같아요. | ||
| // eventPublisher.publishEvent(ReceivedMessageEvent(receiver, saveMessage.id)) // TODO : 생각해보니 이것은 실제로 발송될 때, 전송되어야 하니 나중에 스케줄링 추가하실 때 해당 이벤트 추가해주실 수 있을까요? |
There was a problem hiding this comment.
노노 개발해야됩니다ㅜ 나중에 개발할때 제가 추가하거나 말씀드릴게요!
1. 🔗 관련 이슈
2. 📄 구현한 내용 또는 수정한 내용
알림 관련 기능 완료했어요.
알림을 조회, 읽음처리, 이벤트를 통한 알림 생성과 푸시 알림을 추가했습니다.
3. 🙌 추가적으로 알리고 싶은 내용
기천짱 브랜치 명을 이상하게 했네요 ㅠㅠ 미안해요. 일단 자고, 어디를 리뷰해야 할 지 댓글 남겨놓도록 할게요.
이번에도 테스트 코드가 많을 뿐이지, 프로덕션 코드는 1500줄 정도 될 거에요.
4. 🙄 TODO / 고민하고 있는 것들
5. ✅ 배포 Checklist