Skip to content

Conversation

@soyesenna
Copy link
Contributor

@soyesenna soyesenna commented Nov 14, 2025

Related issue 🛠

작업 내용 💻

  • mail 전송 topic에 recruitmentId 추가

스크린샷 📷

같이 얘기해보고 싶은 내용이 있다면 작성 📢

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 메일 이벤트 처리 시 모집 ID 식별자 추가

@soyesenna soyesenna self-assigned this Nov 14, 2025
@soyesenna soyesenna added the feature New feature or request label Nov 14, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 14, 2025

Walkthrough

RecruitmentUseCase의 sendResultMail 메서드에서 MailSendEvent 빌더에 recruitmentId를 추가하고, MailSendEvent에 새로운 recruitmentId 필드(Long 타입)를 선언하여 메일 이벤트에 모집 식별자를 포함하도록 변경했습니다.

Changes

응집 / 파일(s) 변경 사항
MailSendEvent 필드 확장
src/main/java/com/unionmate/backend/global/kafka/event/MailSendEvent.java
recruitmentId 필드(Long) 추가 (@JsonProperty, @NotNull 포함), NotNull 검증 임포트, 전체 인자 생성자 업데이트
RecruitmentUseCase 이벤트 빌더 수정
src/main/java/com/unionmate/backend/domain/recruitment/application/usecase/RecruitmentUseCase.java
sendResultMail 메서드에서 MailSendEvent 빌더에 recruitmentId(recruitmentId) 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • 주의 영역: MailSendEvent 생성자 서명 변경으로 인한 기존 호출 지점에 미치는 영향 확인 필요

Possibly related issues

Possibly related PRs

Suggested reviewers

  • rootTiket

Poem

🐰 모집 정보를 메일에 담아,
이벤트는 더욱 풍성해지네.
recruitmentId, 작지만 의미 있는 변화,
효율적인 추적을 위해 🎯
한 줄의 코드가 모든 걸 바꾼다네! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 사항의 주요 내용과 완벽하게 일치합니다. 'mail 전송 topic에 recruitmentId 추가'는 이 PR의 핵심 변경을 명확하게 요약합니다.
Description check ✅ Passed PR 설명이 필수 섹션들을 포함하고 있습니다. 관련 이슈와 작업 내용이 기록되었으나, 관련 이슈 번호가 명시되지 않았습니다.
✨ 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 feat/#57

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.

@soyesenna soyesenna changed the title feat: mail 전송 topic에 recruitmentId 추가 Feat/#57 mail 전송 topic에 recruitmentId 추가 Nov 14, 2025
Copy link
Contributor

@rootTiket rootTiket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니당

@soyesenna soyesenna merged commit fb7c7fa into dev Nov 14, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] #57 메일 전송 Topic에 recruitmentId 추가

3 participants