Skip to content

[FEAT] iOS FCM 푸시 알림 지원 추가#364

Merged
hd0rable merged 2 commits into
developfrom
feat/#363-ios-fcm-push-notification
Jul 13, 2026
Merged

[FEAT] iOS FCM 푸시 알림 지원 추가#364
hd0rable merged 2 commits into
developfrom
feat/#363-ios-fcm-push-notification

Conversation

@hd0rable

@hd0rable hd0rable commented Jul 13, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

closes #363

📝 작업 내용

  • PlatformTypeIOS 추가
  • FCM 메시지 빌드 시 ApnsConfig(sound: default) 적용하여 iOS 알림 소리 지원
  • Android/Web 토큰은 FCM이 ApnsConfig 무시하므로 분기 처리 불필요

📸 스크린샷

💬 리뷰 요구사항

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

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

Summary by CodeRabbit

  • 새로운 기능

    • iOS 플랫폼을 알림 대상에 추가했습니다.
    • iOS 푸시 알림에서 기본 알림음이 재생됩니다.
  • 문서

    • 플랫폼 유형 안내에 iOS가 포함되도록 업데이트했습니다.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c5f678a9-84b7-43c7-a5e7-60a9198fe371

📥 Commits

Reviewing files that changed from the base of the PR and between 1503562 and 9c6a449.

📒 Files selected for processing (4)
  • src/main/java/konkuk/thip/message/application/service/FeedNotificationDispatchService.java
  • src/main/java/konkuk/thip/message/application/service/RoomNotificationDispatchService.java
  • src/main/java/konkuk/thip/notification/adapter/in/web/request/FcmTokenRegisterRequest.java
  • src/main/java/konkuk/thip/notification/domain/value/PlatformType.java

Walkthrough

iOS 플랫폼 타입을 추가하고, Feed 및 Room FCM 알림 메시지에 APNs 설정과 기본 사운드를 적용했습니다. 토큰 등록 API 문서에도 IOS 지원 값이 반영되었습니다.

Changes

iOS FCM 알림

Layer / File(s) Summary
iOS 플랫폼 타입 지원
src/main/java/konkuk/thip/notification/domain/value/PlatformType.java, src/main/java/konkuk/thip/notification/adapter/in/web/request/FcmTokenRegisterRequest.java
PlatformTypeIOS를 추가하고 토큰 등록 API의 플랫폼 설명을 갱신했습니다.
FCM 메시지의 APNs 설정
src/main/java/konkuk/thip/message/application/service/*NotificationDispatchService.java
Feed 및 Room 알림 메시지에 ApnsConfig를 추가하고 APNs 사운드를 "default"로 설정했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant FeedNotificationDispatchService
  participant RoomNotificationDispatchService
  participant FirebaseCloudMessaging
  participant IOSDevice
  FeedNotificationDispatchService->>FirebaseCloudMessaging: APNs 설정이 포함된 FCM Message 발송
  RoomNotificationDispatchService->>FirebaseCloudMessaging: APNs 설정이 포함된 FCM Message 발송
  FirebaseCloudMessaging->>IOSDevice: 기본 사운드가 설정된 iOS 알림 전달
Loading

Poem

깡충 토끼가 IOS를 심고
당근처럼 알림 소리도 넣었네.
Feed와 Room이 함께 뛰어
APNs 길을 달려가면,
“딩동!” 사과밭에 울려 퍼져요.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 iOS FCM 푸시 알림 지원 추가라는 주요 변경 사항을 간결하게 잘 요약합니다.
Linked Issues check ✅ Passed #363의 요구사항인 PlatformType IOS 추가와 iOS용 ApnsConfig 설정 추가가 모두 반영되었습니다.
Out of Scope Changes check ✅ Passed 추가된 변경은 iOS 지원과 그에 따른 문서 갱신으로 보이며, 요구사항과 무관한 범위 확장은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#363-ios-fcm-push-notification

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.

@github-actions

Copy link
Copy Markdown

Test Results

488 tests   488 ✅  45s ⏱️
145 suites    0 💤
145 files      0 ❌

Results for commit 9c6a449.

@hd0rable hd0rable merged commit 02b04c2 into develop Jul 13, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] iOS FCM 푸시 알림 지원 추가

1 participant