Skip to content

Comments

feat(recruit-member): memo 신청 시 전화번호 중복 지원자 예외 처리#294

Merged
CSE-Shaco merged 1 commit intoGDGoCINHA:developfrom
CSE-Shaco:develop
Feb 14, 2026
Merged

feat(recruit-member): memo 신청 시 전화번호 중복 지원자 예외 처리#294
CSE-Shaco merged 1 commit intoGDGoCINHA:developfrom
CSE-Shaco:develop

Conversation

@CSE-Shaco
Copy link
Contributor

@CSE-Shaco CSE-Shaco commented Feb 14, 2026

📌 연관된 이슈

ex) #이슈번호, #이슈번호

✨ 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

💬 리뷰 요구사항(선택)

Summary by CodeRabbit

  • New Features
    • 회원 지원 정보 검증이 강화되었습니다. 지원 시 기존 정보 확인 절차가 추가되었으며, 중복된 요청에 대해서는 명확한 오류 메시지를 반환합니다.

@CSE-Shaco CSE-Shaco merged commit 385a7e4 into GDGoCINHA:develop Feb 14, 2026
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Caution

Review failed

The pull request is closed.

걸어가는 과정

RecruitMemberErrorCode에 중복 지원 시 409 CONFLICT 에러코드를 추가하고, RecruitMemberService의 addRecruitMemberMemo 메서드에서 전화번호를 정규화하여 기존 멤버 중복 여부를 검증합니다.

변경 사항

코호트 / 파일(들) 요약
에러 코드 확장
src/main/java/inha/gdgoc/domain/recruit/member/exception/RecruitMemberErrorCode.java
RECRUIT_MEMBER_ALREADY_APPLIED 에러코드 추가 (409 CONFLICT): "이미 지원을 완료하였습니다."
중복 지원 검증 로직
src/main/java/inha/gdgoc/domain/recruit/member/service/RecruitMemberService.java
addRecruitMemberMemo 메서드에서 전화번호 정규화(숫자만 추출) 후 멤버 존재 여부 확인; 존재 시 RECRUIT_MEMBER_ALREADY_APPLIED 예외 발생

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~8분

관련 가능성 있는 PR

🐰 전화번호 깔끔하게 정리하고,
중복 지원 이제는 막아내고,
409로 친절하게 거절하니,
깨끗한 데이터, 반복 방지,
토끼가 웃네요! 🌟

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

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