Skip to content

hotfix: 알림 관련 필드(channelId) 추가#394

Merged
JanooGwan merged 1 commit intodevelopfrom
feature/cam-212
Mar 16, 2026
Merged

hotfix: 알림 관련 필드(channelId) 추가#394
JanooGwan merged 1 commit intodevelopfrom
feature/cam-212

Conversation

@JanooGwan
Copy link
Contributor

🔍 개요


🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

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

@JanooGwan JanooGwan self-assigned this Mar 16, 2026
@JanooGwan JanooGwan added the 리팩토링 리팩터링을 위한 이슈입니다. label Mar 16, 2026
@linear
Copy link

linear bot commented Mar 16, 2026

@JanooGwan JanooGwan merged commit dbfa515 into develop Mar 16, 2026
1 of 3 checks passed
@JanooGwan JanooGwan deleted the feature/cam-212 branch March 16, 2026 13:40
@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 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: e38f01cf-7f45-414d-b209-5e776bf54d25

📥 Commits

Reviewing files that changed from the base of the PR and between efb1cda and 2839c77.

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

📝 Walkthrough

Walkthrough

NotificationService에 새로운 기본 알림 채널 식별자(DEFAULT_NOTIFICATION_CHANNEL_ID)를 도입하고, 채팅, 그룹 채팅 및 일반 알림을 생성할 때 일관되게 사용하도록 업데이트했다. ExpoPushMessage 레코드에 channelId 필드를 추가했다.

Changes

Cohort / File(s) Summary
NotificationService 채널 ID 추가
src/main/java/gg/agit/konect/domain/notification/service/NotificationService.java
DEFAULT_NOTIFICATION_CHANNEL_ID 상수 추가 및 ExpoPushMessage 레코드에 channelId 필드 추가. 모든 ExpoPushMessage 생성 호출 사이트에서 새로운 channelId 파라미터 전달.

Possibly related PRs

  • fix: 딥링크 경로 수정 #252: 두 PR 모두 NotificationService.java의 Expo push 메시지 구성 부분을 수정하며, #252는 push 데이터의 deep-link 경로를 변경하는 한편, 이번 PR은 동일한 push 메시지 구성 사이트에서 channelId 필드를 추가하고 전달한다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 채널 ID를 들고 온 토끼
알림의 길을 정하는 변화
작지만 깔끔한 구조의 향상
한 번에 모든 곳을 정렬하다 ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/cam-212
📝 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.

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

@github-actions
Copy link

⚠️ Checkstyle 위반 사항 발견

Checkstyle 검사에서 코딩 컨벤션 위반이 발견되었습니다.

📋 상세 리포트

@JanooGwan JanooGwan restored the feature/cam-212 branch March 16, 2026 13:46
JanooGwan added a commit that referenced this pull request Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

리팩토링 리팩터링을 위한 이슈입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant