Skip to content

[Refactor] 팀 요청 모집 탭 수정#175

Merged
wonee1 merged 1 commit into
mainfrom
refactor/mypage-168-project
Jun 17, 2026
Merged

[Refactor] 팀 요청 모집 탭 수정#175
wonee1 merged 1 commit into
mainfrom
refactor/mypage-168-project

Conversation

@wonee1

@wonee1 wonee1 commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

🔗 관련 이슈

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

closes #

📌 작업 내용

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

  • 팀 요청 수락 후 확정 버튼을 누르지 않은 PENDING 상태 팀원이 모집중 탭(GET /api/projects?status=RECRUITING)에서 프로젝트를 조회할 수 없는 버그 수정 - ProjectTeamMemberRepository의 쿼리 조건을 = 'CONFIRMED'에서 IN ('CONFIRMED', 'PENDING')으로 변경 변경 파일
    ProjectTeamMemberRepository.java

  • findActiveConfirmedMembershipsWithTeamAndCourse — 내 프로젝트 목록 조회 시 PENDING 상태도 포함

  • findActiveConfirmedMembersWithUserByProjectTeamIds — 프로젝트 카드 팀원 이름 목록도 PENDING 포함

팀 요청 수락 → recruitmentConfirmStatus = PENDING
확정 버튼 클릭 → CONFIRMED

기존 쿼리가 CONFIRMED만 필터링하여, 수락 후 확정 전(PENDING) 팀원은 모집중 탭에서 프로젝트가 보이지 않았음.

🧪 테스트 결과

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

📸 스크린샷 (선택)

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

📎 참고 사항 (선택)

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

Summary by CodeRabbit

  • 개선 사항
    • 팀 구성원 조회 시 승인 대기 중인 구성원도 포함하여 표시되도록 개선되었습니다.

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c61a8c3f-e1f4-46b0-b5c6-02f20e1dbef6

📥 Commits

Reviewing files that changed from the base of the PR and between 41d6ceb and a51d543.

📒 Files selected for processing (1)
  • src/main/java/com/capstone/pickIt/domain/project/repository/ProjectTeamMemberRepository.java

📝 Walkthrough

Walkthrough

ProjectTeamMemberRepository의 두 JPQL 쿼리에서 ptm.recruitmentConfirmStatus 필터 조건을 = 'CONFIRMED' 단일 비교에서 IN ('CONFIRMED', 'PENDING') 복수 포함 조건으로 변경. 메서드 시그니처 및 반환 타입은 유지.

Changes

모집 확정 상태 필터 확장

Layer / File(s) Summary
두 쿼리의 상태 필터 IN 조건으로 확장
src/main/java/com/capstone/pickIt/domain/project/repository/ProjectTeamMemberRepository.java
findActiveConfirmedMembershipsWithTeamAndCoursefindActiveConfirmedMembersWithUserByProjectTeamIds 두 쿼리 모두 ptm.recruitmentConfirmStatus 조건을 = 'CONFIRMED'에서 IN ('CONFIRMED', 'PENDING')으로 변경하여 PENDING 상태 팀원도 결과에 포함.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🔍 CONFIRMED만 보던 눈이,
PENDING까지 품게 되었네.
기다리는 팀원도 이제 보여,
IN 조건 하나로 세상이 넓어졌다. 🌱
작은 쿼리, 큰 변화!

✨ 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 refactor/mypage-168-project

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.

@wonee1 wonee1 merged commit 1207637 into main Jun 17, 2026
2 of 3 checks passed
@wonee1 wonee1 changed the title [refactor] 팀 요청 모집 탭 수정 [Refactor] 팀 요청 모집 탭 수정 Jun 17, 2026
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