Skip to content

pr fix : 단일 게시글 조회 응답 dto 수정 - objectkey, imageurl 추가 - #147

Merged
MINO1020 merged 1 commit into
developfrom
fix-게시글-단일-조회-응답-수정
Mar 9, 2026

Hidden character warning

The head ref may contain hidden characters: "fix-\uac8c\uc2dc\uae00-\ub2e8\uc77c-\uc870\ud68c-\uc751\ub2f5-\uc218\uc815"
Merged

pr fix : 단일 게시글 조회 응답 dto 수정 - objectkey, imageurl 추가#147
MINO1020 merged 1 commit into
developfrom
fix-게시글-단일-조회-응답-수정

Conversation

@MINO1020

@MINO1020 MINO1020 commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #146

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 작업 내용 1
  • 작업 내용 2

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 게시물 이미지 데이터 구조를 개선했습니다. 이미지 정보를 단순 문자열에서 객체 기반으로 변경하여 더 풍부한 메타데이터 관리가 가능해졌습니다.

@MINO1020 MINO1020 linked an issue Mar 9, 2026 that may be closed by this pull request
@coderabbitai

coderabbitai Bot commented Mar 9, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2334596a-c7d3-47a3-acb4-b78cada362d4

📥 Commits

Reviewing files that changed from the base of the PR and between 88d2fd6 and 7d9a21a.

📒 Files selected for processing (4)
  • src/main/java/teamficial/teamficial_be/domain/recruitingPost/dto/RecruitingPostDto.java
  • src/main/java/teamficial/teamficial_be/domain/recruitingPost/dto/response/PostImageResponseDto.java
  • src/main/java/teamficial/teamficial_be/domain/recruitingPost/service/PostImageService.java
  • src/main/java/teamficial/teamficial_be/domain/recruitingPost/service/RecruitingPostService.java

Walkthrough

게시글 단일 조회 응답에서 이미지 데이터 구조를 리팩토링했습니다. 기존의 문자열 기반 이미지 URL 목록을 imageUrl과 objectKey를 포함한 구조화된 DTO로 변경하여 타입 안정성과 데이터 조직성을 개선했습니다.

Changes

Cohort / File(s) Summary
DTO 구조 개편
src/main/java/teamficial/teamficial_be/domain/recruitingPost/dto/RecruitingPostDto.java
RecruitingPostDetailResponseDTO의 imageUrls 필드(List)를 images 필드(List)로 변경하고, from() 팩토리 메서드 시그니처를 업데이트했습니다.
이미지 응답 DTO 추가
src/main/java/teamficial/teamficial_be/domain/recruitingPost/dto/response/PostImageResponseDto.java
imageUrl과 objectKey 필드를 포함하는 새로운 PostImageResponseDto 클래스를 추가했습니다. @Getter, @Builder, @AllArgsConstructor 어노테이션을 사용합니다.
서비스 레이어 업데이트
src/main/java/teamficial/teamficial_be/domain/recruitingPost/service/PostImageService.java, src/main/java/teamficial/teamficial_be/domain/recruitingPost/service/RecruitingPostService.java
PostImageService에 getPostImages(Long postId) 메서드를 추가했으며, 이 메서드는 PostImage 엔티티를 PostImageResponseDto로 변환합니다. RecruitingPostService의 getPost() 메서드를 업데이트하여 새로운 DTO 기반 이미지 구조를 사용하도록 변경했습니다.

Sequence Diagram

sequenceDiagram
    participant Client
    participant RecruitingPostService
    participant PostImageService
    participant Database

    Client->>RecruitingPostService: getPost(postId, userId)
    activate RecruitingPostService
    RecruitingPostService->>Database: 게시글 조회
    Database-->>RecruitingPostService: RecruitingPost
    RecruitingPostService->>PostImageService: getPostImages(postId)
    activate PostImageService
    PostImageService->>Database: PostImage 목록 조회
    Database-->>PostImageService: PostImage entities
    PostImageService->>PostImageService: PostImage → PostImageResponseDto 변환
    Note over PostImageService: imageUrl, objectKey 매핑
    PostImageService-->>RecruitingPostService: List<PostImageResponseDto>
    deactivate PostImageService
    RecruitingPostService->>RecruitingPostService: RecruitingPostDetailResponseDTO 생성
    Note over RecruitingPostService: images 필드에 DTO 목록 할당
    RecruitingPostService-->>Client: RecruitingPostDetailResponseDTO
    deactivate RecruitingPostService
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

🐛 fix

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-게시글-단일-조회-응답-수정

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.

@MINO1020
MINO1020 merged commit 75ef3c0 into develop Mar 9, 2026
1 of 2 checks passed
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.

[fix] 게시글 단일 조회 응답 수정

1 participant