🐛 Fix: 홈 API를 오류를 수정한다 #165

[EunjinWoo]

✨ Related Issue

• 🐛 Fix: 홈 api 수정 #165

---

📌 Task Details

• 홈 요약 조회 오류 수정

---

💬 Review Requirements (Optional)

Summary by CodeRabbit

• 리팩토링
  • 내부 코드 구조 최적화가 적용되었습니다.

---

참고: 이번 변경사항은 주로 내부 구현 개선으로, 사용자에게 직접적인 영향을 미치지 않습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

전체 요약

RecruitmentController에서 사용자 조직의 현재 모집 요약 정보를 조회할 때 메서드 인자 순서를 변경했습니다. recruitmentService.getCurrentSummariesForUser 호출 시 기존의 (organizationId, currentUserId) 순서를 (currentUserId, organizationId) 순서로 수정했습니다.

변경 사항

코호트 / 파일(들)	변경 요약
메서드 인자 순서 변경 src/main/java/KUSITMS/WITHUS/domain/recruitment/recruitment/controller/RecruitmentController.java	getCurrentSummariesForUser 메서드 호출 시 인자 순서를 (organizationId, currentUserId)에서 (currentUserId, organizationId)로 변경

코드 리뷰 예상 난이도

🎯 2 (Simple) | ⏱️ ~5분

시

🐰 한 줄의 변화, 순서를 바꾸니
인자들이 제자리를 찾아가네
작지만 정확한 수정이야
모든 것이 올바르게 흐르리!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 내용과 부분적으로 관련되어 있습니다. '홈 API 오류 수정'은 실제 변경(메서드 인자 순서 변경)의 일부 측면을 나타내지만, 구체적인 수정 내용을 명확하게 설명하지 못합니다.
Description check	✅ Passed	PR 설명이 저장소의 필수 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 상세 내용, 검토 요구사항 섹션을 포함하고 있습니다. 필수 정보가 대부분 작성되어 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 381157b and 45a80d6.

📒 Files selected for processing (1)

• src/main/java/KUSITMS/WITHUS/domain/recruitment/recruitment/controller/RecruitmentController.java

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-docker-image

🔇 Additional comments (1)

src/main/java/KUSITMS/WITHUS/domain/recruitment/recruitment/controller/RecruitmentController.java (1)

104-105: 파라미터 순서 수정이 올바릅니다.

서비스 메서드 시그니처와 일치합니다: getCurrentSummariesForUser(Long userId, Long organizationId) (RecruitmentService.java 라인 23, RecruitmentServiceImpl.java 라인 125)

다른 메서드들과도 일관된 패턴입니다 (예: 라인 27의 getMyDocumentEvaluations(Long userId, Long recruitmentId)).

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[SeongHo5356]

고생하셨습니다~!