Repository navigation
refactor: 통화방 목록 조회 필드 추가 - #38
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis PR refactors room host information representation in room listings. It introduces a new HostInfo DTO to encapsulate host details, updates RoomListResponse with HostInfo and a createdAt timestamp, handles duplicate profile URLs gracefully in MediaService, and optimizes RoomService to batch-fetch host profile URLs instead of fetching them per-room. ChangesRoom Host Information Refactoring
Sequence DiagramsequenceDiagram
participant RoomService
participant RoomRepository
participant MediaService
participant RoomListResponse
RoomService->>RoomRepository: getRooms(page, pageable)
RoomRepository-->>RoomService: Page of Room
RoomService->>RoomService: fetchHostProfileUrls(rooms)
RoomService->>MediaService: getProfileImageUrlMap(distinct hostIds)
MediaService-->>RoomService: Map hostId to profileUrl
RoomService->>RoomListResponse: from(room, participantCount, hostProfileUrl)
RoomListResponse-->>RoomService: RoomListResponse
RoomService-->>RoomService: Page of RoomListResponse
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
통화방 목록 조회 필드 추가
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Chores