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: fcm 데이터 메시지 작성 #102

Merged
merged 9 commits into from
Dec 1, 2024
Merged

Conversation

kimyu0218
Copy link
Collaborator

@kimyu0218 kimyu0218 commented Nov 30, 2024

Issue Number

close: #

작업 개요

데이터 메시지 전송을 위한 테스트 api 작성 (#84)

작업 사항

FCM 메시지 유형

- Notification :  FCM이 자동으로 처리하는 메시지
- Data : 커스텀 동작을 수행할 수 있지만 개발자가 처리해야 하는 메시지
  • 기존에 사용한 Notificationtitlebody 외 추가적인 데이터를 작성할 수 없어 사용자 정의 data를 통해 푸시 알림을 보내야 합니다.
  • send*Message를 data와 구분하기 위해 send*Notification으로 변경했습니다.

고민한 점들(필수 X)

여기에 작성하세요

스크린샷(필수 X)

여기에 작성하세요

@kimyu0218 kimyu0218 requested a review from songyi00 as a code owner November 30, 2024 05:52
@kimyu0218
Copy link
Collaborator Author

  • NotificationData를 구분하기 위해 파일명과 메서드명을 수정했습니다!
  • 기존 PR에는 test api만 작성했지만, 클라이언트 작업을 위해 data로 변경한 로직도 이번 PR에 추가했습니다!

Copy link
Member

@songyi00 songyi00 left a comment

Choose a reason for hiding this comment

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

👍👍

@kimyu0218 kimyu0218 merged commit dfeb29b into develop Dec 1, 2024
1 check passed
@kimyu0218 kimyu0218 deleted the feat/#84-fcm-push-data branch December 1, 2024 21:22
kimyu0218 added a commit that referenced this pull request Dec 2, 2024
* rename: message -> notification

* feat: 테스트용 firebase data 정의

* remove: memberId 필드 삭제

* refactor: notification -> data

* rename: PushNotification -> PushMessage

* fix: 푸시 활성화 여부 확인 시 null 여부도 체크

* test: null 여부 확인 로직 반영

* test: 미션 시작 전/진행 중 푸시 알림 검증 (#104)

* fix: ci 오류 수정
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