Skip to content

Comments

fix: 동아리 지원 알림 딥링크 경로 수정#311

Merged
dh2906 merged 1 commit intodevelopfrom
fix/club-application-notify-deep-link
Feb 23, 2026
Merged

fix: 동아리 지원 알림 딥링크 경로 수정#311
dh2906 merged 1 commit intodevelopfrom
fix/club-application-notify-deep-link

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Feb 23, 2026

🔍 개요

  • close #이슈번호

🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

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

@dh2906 dh2906 self-assigned this Feb 23, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d697cac and 4531f5f.

📒 Files selected for processing (4)
  • src/main/java/gg/agit/konect/domain/club/event/ClubApplicationSubmittedEvent.java
  • src/main/java/gg/agit/konect/domain/club/service/ClubApplicationService.java
  • src/main/java/gg/agit/konect/domain/notification/listener/ClubApplicationNotificationListener.java
  • src/main/java/gg/agit/konect/domain/notification/service/NotificationService.java

📝 Walkthrough

개요

동아리 신청 이벤트와 알림 시스템 전반에 applicationId 매개변수를 추가하여 스레딩합니다. 이벤트 발행, 리스너, 알림 서비스의 메서드 서명과 알림 경로를 함께 업데이트합니다.

변경사항

집단 / 파일 요약
ClubApplicationSubmittedEvent 개선
src/main/java/gg/agit/konect/domain/club/event/ClubApplicationSubmittedEvent.java
applicationId 필드를 추가하고, of() 팩토리 메서드 서명을 업데이트하여 새로운 매개변수를 포함하도록 수정.
이벤트 발행 및 알림 연결
src/main/java/gg/agit/konect/domain/club/service/ClubApplicationService.java, src/main/java/gg/agit/konect/domain/notification/listener/ClubApplicationNotificationListener.java
클럽 신청 생성 시 이벤트 발행에 applicationId를 포함하고, 리스너에서 알림 서비스 호출 시 applicationId를 전달하도록 변경.
NotificationService 메서드 서명 및 경로 업데이트
src/main/java/gg/agit/konect/konect/domain/notification/service/NotificationService.java
sendClubApplicationSubmittedNotification 메서드에 applicationId 매개변수를 추가하고, 알림 경로를 "clubs/" + clubId + "/applications"에서 "mypage/manager/" + clubId + "/applications/" + applicationId로 변경.

추정 검토 난이도

🎯 2 (단순) | ⏱️ ~10분

관련 가능성이 있는 PR

제안 레이블

버그

🐰 신청서에 아이디를 담아,
알림의 길을 똑똑히 찾아,
마이페이지로 쏙 들어가네,
매개변수 춤을 맞추며,
더욱 정확한 알림이 춤을 춘다! 🎉

✨ 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 fix/club-application-notify-deep-link

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

@dh2906 dh2906 merged commit 8878073 into develop Feb 23, 2026
1 of 2 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.

1 participant