Skip to content

Conversation

@Kim-TaeUk
Copy link
Member

@Kim-TaeUk Kim-TaeUk commented Jan 23, 2025

Related Issue

Key Changes

  • 멀티 디바이스를 고려
    • UserDevice로 엔티티 분리
    • 기존 sendPushMessagesendMulticastPushMessage로 변경: 유저 한 명에게 발송하더라도 기기가 여러 대면 fcmToken이 여러 개가 되기 때문
  • 푸시 알림 수신 여부 설정 및 수정을 위한 api 추가
  • fcm token 등록 api -> 등록 및 수정 api로 변경

To Reviewers

  • 운영 서버에서 isPushEnabled 컬럼 추가할 때 문제 발생 가능성

Copy link
Member

@ChaeAg ChaeAg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고 많으셨어요 ㅎㅎ!!

- 유저 당 notification은 1개
- 유저의 디바이스에 모두 푸시 보내기
@Kim-TaeUk Kim-TaeUk requested a review from ChaeAg January 23, 2025 16:54
@ChaeAg ChaeAg merged commit 29feb73 into dev Jan 24, 2025
1 check 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.

[FIX] 누락된 푸시알림 구현 추가

3 participants