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

fix: 미션 삭제 푸시 알림 #114

Merged
merged 2 commits into from
Dec 22, 2024
Merged

fix: 미션 삭제 푸시 알림 #114

merged 2 commits into from
Dec 22, 2024

Conversation

kimyu0218
Copy link
Collaborator

Issue Number

close: #113

작업 개요

  • 미션 삭제 시 호스트 여부와 관계 없이 모든 미션 멤버에게 삭제 푸시 알림이 전송된다.
  • 내가 삭제한 미션에 대해 알림이 가는 게 어색하다고 판단하여 호스트에게는 삭제 푸시 알림을 보내지 않도록 수정한다.

작업 사항

삭제 푸시 알림을 전송하기 전에 호스트에 대해 해당 미션을 구독 취소했습니다.

+ deviceSubscriptionService.unsubscribeFromDeletedMissionForHost(event.memberId(), event.missionId());
pushMessageSender.sendGroupNotificationWithData(
    MISSION_DELETED.getTitle(),
    MISSION_DELETED.getBody(),
    data,
    topic
);

고민한 점들(필수 X)

여기에 작성하세요

스크린샷(필수 X)

여기에 작성하세요

@kimyu0218 kimyu0218 requested a review from songyi00 as a code owner December 22, 2024 08:52
@kimyu0218 kimyu0218 merged commit 5484604 into develop Dec 22, 2024
1 check passed
@kimyu0218 kimyu0218 deleted the fix/#113-delete-push branch December 22, 2024 08:56
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.

미션 삭제 시 호스트에게 푸시 알림을 보내지 않는다
1 participant