Skip to content

main -> develop#402

Merged
dh2906 merged 6 commits intodevelopfrom
main
Mar 18, 2026
Merged

main -> develop#402
dh2906 merged 6 commits intodevelopfrom
main

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Mar 18, 2026

🔍 개요

  • main 브랜치와 develop 브랜치의 히스토리 일치를 위한 PR입니다.

🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

JanooGwan and others added 6 commits March 12, 2026 16:36
hotfix: 알림 관련 필드 추가
* fix: EXPO 알림 타임아웃 재시도 추가

* refactor: 롬복 기반 생성자 어노테이션으로 교체

* fix: EXPO 알림 실패 원인 로그를 예외별로 구체화

* fix: pre-push 포맷 대상 경로를 절대경로로 정규화

* chore: 코드 포맷팅
@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4257d90e-9444-406d-b8dc-dea8716a61e3

📥 Commits

Reviewing files that changed from the base of the PR and between b64d4c3 and 04c1196.

📒 Files selected for processing (1)
  • src/main/java/gg/agit/konect/domain/notification/service/NotificationService.java

📝 Walkthrough

걸어보기

NotificationService에 새로운 알림 채널 상수를 추가하고, 모든 알림 전송 메서드를 업데이트하여 ExpoPushMessage에 channelId를 전달하도록 수정했습니다.

변경사항

변경 대상 / 파일 요약
알림 채널 ID 추가
src/main/java/gg/agit/konect/domain/notification/service/NotificationService.java
기본 알림 채널 상수(DEFAULT_NOTIFICATION_CHANNEL_ID)를 추가하고, 모든 알림 전송 메서드에서 ExpoPushMessage 생성 시 channelId 필드를 포함하도록 업데이트.

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10분

관련 가능성이 있는 PR

  • hotfix: 알림 관련 필드(channelId) 추가 #394: 동일한 NotificationService와 ExpoPushMessage 레코드에 channelId/DEFAULT_NOTIFICATION_CHANNEL_ID 필드를 추가하고 전달하는 변경사항으로 직접적으로 관련됨.
  • fix: 딥링크 경로 수정 #252: NotificationService.java의 채팅 알림 플로우(sendChatNotification/sendGroupChatNotification)를 수정하며, 해당 PR은 깊은 링크 경로를 변경하므로 연관됨.

제안 라벨

리팩토링

🐰 채널 ID를 새로이 담아,
모든 메시지에 실어 보내네.
상수로 정의하고 스레드는 안전하게,
알림은 더욱 질서정연하게 흐르고,
토끼도 만족한 깔끔한 변화! 🎉

✨ 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 main
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@dh2906 dh2906 merged commit 3cbd316 into develop Mar 18, 2026
4 of 5 checks 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.

2 participants