feat: 프로젝트 참여 멤버 조회 api 구현 - #19
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (15)
💤 Files with no reviewable changes (6)
📝 WalkthroughWalkthrough프로젝트 멤버 목록·상세 조회 API와 접근 권한 검증을 추가했습니다. 동시에 프로젝트 및 포트폴리오의 Changes프로젝트 멤버 조회 및 접근 검증
프로젝트
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant Client
participant ProjectMemberController
participant ProjectMemberService
participant ProjectAccessValidator
participant ProjectMemberRepository
participant ProjectUserRoleRepository
Client->>ProjectMemberController: X-USER-ID와 projectId 전달
ProjectMemberController->>ProjectMemberService: 멤버 목록 또는 상세 조회 호출
ProjectMemberService->>ProjectAccessValidator: 프로젝트 접근 검증
ProjectMemberService->>ProjectMemberRepository: 활성 멤버 조회
ProjectMemberService->>ProjectUserRoleRepository: 멤버 역할 조회
ProjectMemberService-->>ProjectMemberController: 멤버 응답 DTO 반환
ProjectMemberController-->>Client: ApiResponse 성공 응답
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
Comment |
young0206
approved these changes
Jul 19, 2026
This was referenced Jul 19, 2026
Merged
Merged
Merged
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 관련 이슈 (Related Issue)
📝 작업 내용
프로젝트 참여 목록과 멤버 상세 카드를 조회할 수 있도록 프로젝트 멤버 조회 API를 추가했습니다.
또한 프로젝트 접근 권한 검증 로직을
ProjectAccessValidator로 분리하고, 프로젝트 커스텀 유형 입력 필드를 제거했습니다.주요 검토 파일
주요 변경 파일
src/main/java/com/slatto/domain/project/controller/ProjectMemberController.java- 프로젝트 멤버 목록/상세 조회 엔드포인트 추가src/main/java/com/slatto/domain/project/service/ProjectMemberService.java- 프로젝트 멤버 조회 비즈니스 로직 추가src/main/java/com/slatto/domain/project/service/ProjectAccessValidator.java- 프로젝트 접근/관리자 권한 검증 로직 분리src/main/java/com/slatto/domain/project/dto/ProjectMember*Response.java- 프로젝트 멤버 조회 응답 DTO 추가1. 프로젝트 멤버 목록 조회 API 추가
프로젝트 참여 목록 화면에서 필요한 멤버 목록을 조회할 수 있도록 API를 추가했습니다.
2. 프로젝트 멤버 상세 카드 조회 API 추가
작성자/참여자 카드 화면에서 필요한 멤버 상세 정보를 조회할 수 있도록 API를 추가했습니다.
상세 응답에는 목록 응답 필드에 더해 이메일과 소개 정보를 포함했습니다.
3. 프로젝트 접근 권한 검증 로직 분리
프로젝트 접근 권한과 관리자 권한 검증을 여러 서비스에서 재사용할 수 있도록
ProjectAccessValidator로 분리했습니다.4. 커스텀 프로젝트 유형 입력 필드 제거
pm님과 얘기했던 대로 프로젝트 생성/수정 요청과 엔티티에서
customTypeName을 제거했습니다.프로젝트 유형은 enum 값만 사용하도록 맞췄습니다.
5. 임시 인증 헤더 유지
아직 JWT 인증 구조가 연결되지 않았기 때문에, 기존 프로젝트 API와 동일하게 임시로
X-USER-ID헤더에서 사용자 ID를 받아 처리하도록 구성했습니다.X-USER-ID: 1추후 인증 모듈이 연결되면
@AuthenticationPrincipal또는 공통 인증 유틸 기반으로 교체할 예정입니다.✅ PR 체크리스트
./gradlew compileJava로 컴파일을 확인했습니다.테스트 참고
./gradlew compileJava는 성공했습니다../gradlew test는 기존SlattoApplicationTests에서 JPA 자동설정을 제외한 상태로 전체 Spring Context를 띄우고 있어, Repository bean을 찾지 못하는 문제가 있을 수 있습니다.💬 To Reviewers
Summary by CodeRabbit
새 기능
변경 사항