Skip to content

#396 [Fix] Android FCM 푸시에 title/body가 실리지 않아 알림이 안 뜨던 문제 수정 - #397

Merged
si-zero merged 1 commit into
devfrom
fix/#396
Aug 16, 2026
Merged

#396 [Fix] Android FCM 푸시에 title/body가 실리지 않아 알림이 안 뜨던 문제 수정#397
si-zero merged 1 commit into
devfrom
fix/#396

Conversation

@si-zero

@si-zero si-zero commented Aug 16, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

🐛 Fix

내용 파일
FcmPushService.send()가 title/body 파라미터를 무시하고 data map에 안 실어 Android에서 알림이 안 뜨던 문제 수정 (data map에 title, body 추가) FcmPushService.java

📌 공유 사항

  1. iOS(ApnsPushService)는 setAlertTitle/setAlertBody로 원래 정상 전달되고 있었음. Android만 영향받던 버그.
  2. 신규 배틀 알림, 댓글 좋아요/새 댓글 알림 등 Android로 가는 기존 푸시 전부 이 수정으로 함께 정상화됨.

✅ 체크리스트

  • Reviewer에 팀원들을 선택했나요?
  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 팀원들에게 PR 링크 공유를 했나요?

📸 스크린샷

FcmPushServiceTest 단위 테스트로 Message data payload에 title/body가 포함되는지 검증 완료 (테스트 통과).

💬 리뷰 요구사항

  1. data-only 메시지 구조를 유지한 채 data map에 title/body를 추가하는 방식으로 고쳤습니다. FCM notification 필드를 별도로 쓰는 방식 대신 이 방식이 맞는지 확인 부탁드립니다.

@si-zero si-zero self-assigned this Aug 16, 2026
@si-zero si-zero added the 🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다. label Aug 16, 2026
@si-zero
si-zero merged commit 2e72bf7 into dev Aug 16, 2026
1 check passed
@si-zero
si-zero deleted the fix/#396 branch August 16, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Fix: Android FCM 푸시에 title/body가 실리지 않아 알림이 안 뜨는 버그 수정

1 participant