Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | Summary |
|---|---|
FCM 네비게이션 열거형 backend/src/main/java/moadong/fcm/enums/FcmAction.java |
웹뷰 네비게이션 목적의 새 enum FcmAction 추가 (NAVIGATE_WEBVIEW). |
알림 데이터 페이로드 구성 backend/src/main/java/moadong/club/util/RecruitmentStateCalculator.java |
buildNotificationData(Club) 공용 메서드 추가. recruitment 메시지 생성 시 putAllData(buildNotificationData(club))로 path(/webview/clubDetail/{id}), action(NAVIGATE_WEBVIEW), clubId 포함한 데이터 페이로드를 메시지에 첨부하도록 변경. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~8 minutes
🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | Write docstrings for the functions missing them to satisfy the coverage threshold. |
✅ Passed checks (4 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | PR 제목은 'FCM 알림 페이로드에 네비게이션용 데이터 추가'라는 주요 변경사항을 명확하게 설명하고 있습니다. |
| Linked Issues check | ✅ Passed | PR의 코드 변경사항이 MOA-630의 요구사항을 충족합니다: FcmAction 열거형 추가, RecruitmentStateCalculator에 buildNotificationData 메서드 추가, 네비게이션 데이터(path, action, clubId) 포함. |
| Out of Scope Changes check | ✅ Passed | 모든 코드 변경사항이 MOA-630의 FCM 알림 네비게이션 데이터 추가 목표와 직접적으로 관련되어 있습니다. |
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ 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
feature/#1174-add-noti-navigation-MOA-630
Tip
Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
Test Results72 tests 72 ✅ 20s ⏱️ Results for commit 25a77f8. ♻️ This comment has been updated with latest results. |
|
이 형식으로 봐주세요 |
#️⃣연관된 이슈
#1174
📝작업 내용
네비게이션용 데이터 추가했습니다
다만 스케쥴링 코드가 테스트 용이하게 코드가 짜져있지않아 리팩토링 필요성이 느껴져 지라에 태스크 추가했습니다.
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit