Skip to content

Conversation

@david-parkk
Copy link
Contributor

@david-parkk david-parkk commented Dec 13, 2025

#️⃣ 연관된 이슈

closes #360

📝작업 내용

  • Fix FCM Gradle
  • Fix FCM Function

작업 상세 내용

상세 내용을 입력해주세요.

💬리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

Summary by CodeRabbit

릴리스 노트

  • 의존성 업그레이드

    • Firebase Admin SDK를 버전 9.3.0으로 업그레이드했습니다.
  • 개선 사항

    • FCM 서비스의 메시지 전송 방식을 최적화했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 13, 2025

Walkthrough

Firebase Admin SDK가 6.8.1에서 9.3.0으로 업그레이드되었으며, FCM 멀티캐스트 메시지 전송 API가 sendMulticast()에서 sendEachForMulticast()로 변경되었습니다. 에러 처리 로직은 유지됩니다.

Changes

코호트 / 파일 변경 요약
Firebase Admin SDK 의존성 업그레이드
build.gradle
firebase-admin 버전을 6.8.1에서 9.3.0으로 업데이트
FCM 멀티캐스트 API 업데이트
src/main/java/ku_rum/backend/domain/alarm/application/FcmService.java
firebaseMessaging.sendMulticast(message)firebaseMessaging.sendEachForMulticast(message) 메서드 호출 변경

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~15 분

  • Firebase Admin SDK 9.3.0의 sendEachForMulticast() 메서드가 기존 sendMulticast()와 호환되는지 확인 필요
  • 멀티캐스트 메시지 전송의 반환 값과 에러 처리 로직이 기존과 일치하는지 검증 필요

관련 PR

  • [Fix] FCM 로깅 #359: FcmService.java의 예외 처리 로직에 SLF4J 로깅 추가로 동일 파일 수정
  • [Fix] FCM Gradle #360: Firebase Admin SDK 의존성 버전 업그레이드로 동일한 build.gradle 수정

Poem

🐰 방기니의 작은 목소리로...

파이어베이스 업그레이드, 아홉 점 삼으로 쌩!
멀티캐스트는 이제 각각 보내 웅~
안정성 높아졌네, 우리 앱도 반짝반짝 ✨
래빗이 축하해, 준비 완료! 🚀

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ❓ Inconclusive 설명이 기본 템플릿을 따르고 있으나, '작업 상세 내용' 섹션이 빈 플레이스홀더로 남아있어 구체적인 설명이 부족합니다. '작업 상세 내용' 섹션을 구체적으로 작성하여 각 변경사항의 목적과 영향을 명확히 설명해주세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 주요 변경사항을 명확하게 요약하고 있습니다: Firebase Admin SDK 버전 업그레이드 및 FCM 함수 수정.
Linked Issues check ✅ Passed PR의 코드 변경사항이 링크된 이슈 #360의 요구사항을 충족합니다: Firebase Gradle 버전 업그레이드 및 FCM 함수 수정이 모두 구현되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 링크된 이슈 범위 내에 있습니다: build.gradle의 Firebase Admin SDK 버전 업그레이드와 FCM 함수의 sendMulticast 호출 변경이 명확히 정의된 작업과 일치합니다.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#358

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bfdd292 and a483d78.

📒 Files selected for processing (2)
  • build.gradle (1 hunks)
  • src/main/java/ku_rum/backend/domain/alarm/application/FcmService.java (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

Test Results

 37 files   37 suites   11s ⏱️
150 tests 150 ✅ 0 💤 0 ❌
151 runs  151 ✅ 0 💤 0 ❌

Results for commit 363c2db.

@david-parkk david-parkk merged commit 9986371 into develop Dec 13, 2025
3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Dec 16, 2025
1 task
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.

3 participants