refactor: 유저 정보 조회 수정#155
Hidden character warning
Conversation
|
Caution Review failedThe pull request is closed. Walkthrough사용자 프로필 조회 시, 리포지토리에 사용자-정원-아바타-아바타마스터를 즉시 로딩하는 메서드를 추가하고, 서비스에서 프로필 이미지 결정 로직을 정원의 최소 slotNumber 기반으로 재구성하여 아바타/마스터/사용자 이미지 URL 순으로 폴백하도록 변경했습니다. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant C as Client
participant S as UserService
participant R as UserRepository
C->>S: getProfile(userId, profileUserId)
S->>R: findByIdWithGardensAndAvatars(profileUserId)
R-->>S: User{gardens{avatar{avatarMaster}}}
rect rgba(235, 245, 255, 0.8)
note right of S: 프로필 이미지 결정 로직 변경
S->>S: 최소 slotNumber 정원 선택
alt Avatar.imageUrl 존재
S->>S: 이미지 = Avatar.imageUrl
else AvatarMaster 기본 이미지
S->>S: 이미지 = AvatarMaster.defaultImageUrl
else 아바타 없음
S->>S: 이미지 = user.profileImageUrl
end
end
S-->>C: ProfileResponse(선택된 이미지 포함)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (2)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
📝 개요
이번 PR의 핵심 내용을 한 줄로 요약해 주세요.
💻 작업 내용
이번 PR에서 작업한 내용을 상세히 설명해 주세요.
작업 내용 1
작업 내용 2
...
✅ PR 체크리스트
PR을 보내기 전에 아래 체크리스트를 확인해 주세요.
커밋 메시지는 포맷에 맞게 작성했나요?
스스로 코드를 다시 한번 검토했나요?
관련 이슈를 연결했나요?
빌드 및 테스트가 로컬에서 성공했나요?
🔗 관련 이슈
이번 PR과 관련된 이슈 번호를 기재해 주세요. 예: Closes #154
스크린샷 (선택)
UI 변경 사항이 있다면 스크린샷을 첨부해 주세요.
Summary by CodeRabbit