Repository navigation
[feature/#727] 게임판 알림 기능 구현 - #736
Merged
Merged
Conversation
- NotificationResourceType.GAME_BOARD, NotificationSource.GAME 추가 - NotificationOutboxEventType에 GAME_STARTED, GAME_HOST_ASSIGNED 추가 - NotificationMessageGenerator에 게임 시작/진행자 지정 메시지 추가
- GameStartedEvent 추가 (게임 시작 시 배정 회원 대상, 게스트 제외) - GameNotificationStrategy 추가: GAME_STARTED -> GAME_BOARD destination 변환 - NotificationOutboxEventListener에 게임 시작 이벤트 핸들러 배선 - GameCommandService.startGame에서 회원 수집 후 이벤트 발행 - 코트 이름(사용자 지정/기본값)을 알림 문구에 사용 - 전략/발행 단위 테스트 추가
Dimo-2562
reviewed
Sep 5, 2026
Dimo-2562
left a comment
Dimo-2562
left a comment
Contributor
There was a problem hiding this comment.
FcmService:64-70을 보면 현재 푸시 알림에서 단순 title와 content만 적혀 있습니다.
프론트에서 여러 종류의 푸시 알림마다 이동되는 페이지가 다를텐데 혹시 알림의 종류를 주는 건 어떨까요?
| ExerciseErrorCode.INVALID_GAME_HOST_CANDIDATE)); | ||
|
|
||
| exercise.changeGameHost(command.participantId()); | ||
| publishGameHostAssigned(exercise, command.participantId()); |
Contributor
There was a problem hiding this comment.
현재 게임 지정자 변경을 할 때 자기 자신으로 유지하는 경우에도 성공을 보장하고 있습니다.
하지만 이 경우 자기자신으로 유지했음에도 변경 이벤트가 발행되어 푸시 알림이 갈 우려가 있으므로 이벤트 발행을 조건부로 하는 건 어떨까요??
| return; | ||
| } | ||
|
|
||
| Exercise exercise = exerciseRepository.findByGameBoardId(gameBoardId) |
Contributor
There was a problem hiding this comment.
별건 아닌데 startGame()의 gameBoardAccessValidator.validateGameHost()에서 exercise를 조회함에도 푸시 알림을 위해 다시 조회를 진행하여 중복이 있는 것 같습니다!
| * @param imageKey 알림 이미지 키 (모임 이미지, 없으면 null) | ||
| * @param recipientMemberId 알림 수신 대상 회원 ID (새로 지정된 진행자 본인) | ||
| */ | ||
| public record GameHostAssignedEvent( |
Contributor
There was a problem hiding this comment.
gameHostId의 소유권이 exercise 도메인에 있으므로 gameHostAssignedEvent를 exercise 도메인으로 옮기는 방향은 어떨까요?
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❤️ 기능 설명
게임판 기능 추가에 따라 게임 관련 푸시/인앱 알림 2종을 구현했습니다. 두 알림 모두 클릭 시 게임판 페이지(GAME_BOARD)로 이동합니다.
연결된 issue
연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.
close #727
🩷 Approve 하기 전 확인해주세요!
✅ 체크리스트