Skip to content

feat(member): 추가 프로필 이미지 조회 기능 추가 - #448

Merged
stemmmm merged 1 commit into
developfrom
feat/445
Jun 16, 2026
Merged

feat(member): 추가 프로필 이미지 조회 기능 추가#448
stemmmm merged 1 commit into
developfrom
feat/445

Conversation

@stemmmm

@stemmmm stemmmm commented Jun 16, 2026

Copy link
Copy Markdown
Contributor
  • 추가 프로필 이미지 목록 필드 도입
  • 메인 이미지 제외한 추가 이미지 필터링 및 매핑 로직 구현
  • 관련 유닛 테스트 추가 및 검증

Summary by CodeRabbit

  • New Features

    • 멤버 프로필에 메인 이미지 외에 추가 프로필 이미지를 표시할 수 있도록 개선했습니다.
    • 추가 이미지는 설정된 순서에 따라 정렬되어 표시됩니다.
  • Tests

    • 추가 프로필 이미지 조회 및 표시 로직의 테스트 케이스를 추가했습니다.

- 추가 프로필 이미지 목록 필드 도입
- 메인 이미지 제외한 추가 이미지 필터링 및 매핑 로직 구현
- 관련 유닛 테스트 추가 및 검증

#445
@stemmmm stemmmm self-assigned this Jun 16, 2026
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

ProfileImageInfo DTO 레코드를 신규 추가하고, 기존 MemberInfo 레코드에 additionalProfileImages 컴포넌트(List<ProfileImageInfo>)를 추가했다. MemberQueryServiceProfileImageQueryRepository를 주입해 getMemberProfile 호출 시 대상 멤버의 이미지를 조회하고, MemberMapper.toBasicInfo()의 시그니처를 List<ProfileImageView> 파라미터를 추가로 받도록 변경했다. Mapper에서는 primary가 아닌 이미지만 필터링해 ProfileImageInfo로 변환한 뒤 MemberInfo 생성자에 전달한다. 관련 테스트 2개를 추가했다.

추정 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Description check ❓ Inconclusive Pull request description is minimal and lacks detail about implementation approach, rationale, and alignment with repository template requirements. 구체적인 구현 방식, 변경 배경, 그리고 사용 시나리오에 대한 설명을 추가하고 템플릿 섹션(참고 자료, 노트 등)을 활용하여 문서화의 완성도를 높이세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed Pull request title clearly summarizes the main change: adding functionality to retrieve additional profile images for members.
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 feat/445

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.

@stemmmm
stemmmm merged commit c61fa47 into develop Jun 16, 2026
2 of 3 checks passed
@stemmmm
stemmmm deleted the feat/445 branch June 16, 2026 11:21
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