Skip to content

[#14] 경매 상세 조회 기능 - #24

Merged
JIWON27 merged 3 commits into
mainfrom
feature/14
Aug 29, 2026
Merged

JIWON27 merged 3 commits into
mainfrom
feature/14

Conversation

@JIWON27

@JIWON27 JIWON27 commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

개요

경매 상세 조회 기능 구현

  • AuctionService에 getDetail 메서드 추가
    • 존재하는 경매는 상세 정보 응답, 존재하지 않으면 CommonErrorCode.NOT_FOUND 예외
  • GET /api/v1/auctions/{auctionId} 엔드포인트 추가 (비로그인 사용자도 접근 가능하도록 SecurityConfig에 permitAll 반영)
  • 응답 DTO로 AuctionDetailResponse 신규 작성
  • CameraImageRepository에 findByCameraId(Long cameraId) 추가
  • 경매 상세 조회 기능 서비스/컨트롤

메인 리뷰어 지정 및 Due Date

  • 메인 리뷰어 : blue , Due Date : 2026-08-30

리뷰 시 참고 사항

  • User 엔티티에 nickname 필드가 없어서, AuctionDetailResponse.sellerNickname을 "판매자"+sellerId 형태의 임시값으로 채웠습니다. 코드에 TODO로 남겨뒀는데, 닉네임 필드가 추가되면 변경하도록 하겠습니다
  • imageKeys는 현재 파일명을 그대로 내려줍니다. 다운로드 가능한 실제 경로 제공은 FileStorage 쪽 기능 확장이 필요해서 이번 범위에서 제외했습니다. 요것도 TODO로 남겨놨습니다.
  • 상세 조회는 비로그인 사용자도 볼 수 있도록 SecurityConfig에 GET /api/v1/auctions/** permitAll을 추가했습니다.

TODO

  • User 엔티티에 nickname 필드가 추가되면 AuctionDetailResponse의 임시 닉네임 생성 로직 제거
  • 이미지 다운로드 가능한 경로(URL) 제공 기능 구현
  • 경매 목록 조회(검색/필터/정렬/페이지네이션) 기능 구현

References

[1] x

체크리스트

  • PR 제목을 명령형으로 작성했습니다.
  • PR을 연관되는 github issue에 연결했습니다.
  • 리뷰 리퀘스트 전에 셀프 리뷰를 진행했습니다
  • 변경사항에 대한 테스트코드를 추가했습니다. 또는, 테스트 코드가 필요없는 이유가 있습니다.

@JIWON27
JIWON27 requested a review from blue000927 August 26, 2026 16:55
@JIWON27 JIWON27 self-assigned this Aug 26, 2026
@JIWON27 JIWON27 linked an issue Aug 26, 2026 that may be closed by this pull request
Comment on lines +12 to +17
public record AuctionDetailResponse(
Long id,
Long cameraId,
CameraCategory category,
String brand,
String modelName,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(minor) Response 안에 필드가 생각보다 많은데 관련성 있는 필드를 묶어서 별도 nested dto로 반환해보는건 어때요?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 ! 관련있는 필드들 묶도록 하겠습니다

@sonarqubecloud

Copy link
Copy Markdown

@JIWON27
JIWON27 merged commit 668ccca into main Aug 29, 2026
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.

[Task] 경매 상세 조회 기능

2 participants