Skip to content

[Fix] 프로젝트 상세 조회에서 멤버 학년도 조회하도록 수정#167

Merged
ownue merged 1 commit into
mainfrom
fix/get-project-member
Jun 3, 2026
Merged

[Fix] 프로젝트 상세 조회에서 멤버 학년도 조회하도록 수정#167
ownue merged 1 commit into
mainfrom
fix/get-project-member

Conversation

@ownue

@ownue ownue commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

🔗 관련 이슈

관련된 이슈 번호를 적어주세요.

X

📌 작업 내용

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

  • 프로젝트 상세 조회에서 멤버 학년도 조회하도록 수정

🧪 테스트 결과

Postman 스크린샷, 테스트 통과 여부 등을 첨부해주세요.

📸 스크린샷 (선택)

필요시 스크린샷을 첨부해주세요.

📎 참고 사항 (선택)

리뷰어에게 전달할 내용이 있다면 작성해주세요.

Summary by CodeRabbit

릴리즈 노트

  • 개선 사항
    • 프로젝트 팀 멤버 정보 조회 응답에 학년 정보가 추가되었습니다.

@ownue ownue self-assigned this Jun 3, 2026
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c1bd5275-a306-4ea5-8cd3-d2fbe2a30334

📥 Commits

Reviewing files that changed from the base of the PR and between 1a83007 and 8a8d496.

📒 Files selected for processing (2)
  • src/main/java/com/capstone/pickIt/api/project/converter/ProjectConverter.java
  • src/main/java/com/capstone/pickIt/api/project/dto/response/ProjectMemberSummaryDTO.java

📝 Walkthrough

Walkthrough

프로젝트 팀 멤버의 요약 정보 응답에 사용자의 학년(grade) 필드를 추가했습니다. DTO 레코드에 Integer grade 컴포넌트를 정의하고, 변환기에서 사용자의 학년 데이터를 함께 전달하도록 업데이트됨.

Changes

멤버 응답 DTO에 학년 필드 포함

Layer / File(s) Summary
프로젝트 멤버 요약 DTO 학년 필드 추가
src/main/java/com/capstone/pickIt/api/project/dto/response/ProjectMemberSummaryDTO.java
ProjectMemberSummaryDTO 레코드의 컴포넌트에 Integer grade 필드를 String major 뒤에 추가하여 응답 계약을 확장함.
컨버터에서 학년 값 전달
src/main/java/com/capstone/pickIt/api/project/converter/ProjectConverter.java
ProjectConverter.toMemberSummary() 메서드에서 member.getUser().getGrade()를 생성자 인자로 전달하여 새 필드를 채우도록 구현함.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

한 줄의 grade, 두 파일에 담아 ✨
DTO에서 정의하고 변환기에서 실어
멤버의 학년이 이제 우리 응답에 🎓
간단하고 명확한 데이터 확장!

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 정확하게 반영하고 있습니다. 프로젝트 상세 조회에서 멤버 학년도 정보를 조회하도록 추가한 변경 사항을 명확하게 설명합니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/get-project-member

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


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.

@ownue ownue merged commit 39173d8 into main Jun 3, 2026
3 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