Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat : 반복미션 시작 알림 기능 추가 #195

Merged
merged 3 commits into from
Jan 7, 2024
Merged

feat : 반복미션 시작 알림 기능 추가 #195

merged 3 commits into from
Jan 7, 2024

Conversation

seungueonn
Copy link
Member

@seungueonn seungueonn commented Jan 7, 2024

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 업데이트
  • 기타 사소한 수정

개요

  • 반복미션 시작 알림
    • 일요일에서 월요일 넘어가는 밤 12시에 반복미션 시작할 때 그 미션에 해당하는 모든 인원에게 알림 발송
    • 제가 깜빡하고 안만들었답니다 ㅎㅎ

변경 사항

  • 기존 스케쥴링에 미션 상태를 바꾸기 전에 알림 전송 로직을 수행합니다!
  • 크흠 조금 야매 .. 느낌이 없지않아 있는데요 .. 기존 미션 생성 알림 메소드를 활용하였습니다.teamMemberGetService.getNewUploadInfo(team.getTeamId(), member.getId()); 는 미션 생성한 멤버를 제외하고 가져왔다면, 현재는 모든 모임원이 알림을 받아야하니 .. teamMemberGetService.getNewUploadInfo(team.getTeamId(), 0L); 로 사용하였습니다 ㅎㅎ

코드 리뷰 시 참고 사항

@Scheduled(cron = "0 0 0 * * MON")
public void RepeatMissionStart() {
List<Mission> startMission = missionQueryService.findRepeatMissionByStatus(MissionStatus.WAIT);
startMission.forEach(
mission -> {
// 미션 시작 알림
sendMissionStartAlarmUseCase.sendRepeatMissionStartAlarm(mission);
mission.updateStatus(MissionStatus.ONGOING);
}
);
}

Optional<List<NewUploadInfo>> newUploadInfos=teamMemberGetService.getNewUploadInfo(team.getTeamId(), 0L);
Optional<List<MemberIdAndToken>> memberIdAndTokensByPush = AlarmHistoryMapper.getNewUploadPushInfo(newUploadInfos);
Optional<List<MemberIdAndToken>> memberIdAndTokensBySave = AlarmHistoryMapper.getNewUploadSaveInfo(newUploadInfos);
// 알림 보내기
eventPublisher.publishEvent(new MultiFcmEvent(title, message, memberIdAndTokensByPush, memberIdAndTokensBySave, createIdInfo(team.getTeamId(), mission.getId(),mission.getType(),mission.getStatus()), team.getName(), AlarmType.NEW_UPLOAD, PagePath.MISSION_PATH.getValue()));
}

테스트 결과

@minsu20 minsu20 merged commit 8d828ea into release Jan 7, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants