Skip to content

Comments

[Refator] 책장 관련 API 수정#323

Merged
U-hee merged 5 commits intodevfrom
refator/add-book-list-field
Feb 13, 2026
Merged

[Refator] 책장 관련 API 수정#323
U-hee merged 5 commits intodevfrom
refator/add-book-list-field

Conversation

@U-hee
Copy link
Contributor

@U-hee U-hee commented Feb 11, 2026

PR 요약

타임라인의 사전의견 삭제 연동을 위해 PRE_OPINION 응답에 gatheringId, meetingId를 추가하고, 잘못 확장된 topic-answer 응답 정의는 원복했습니다.

  • 기능 추가
  • 버그 수정
  • 코드 리팩토링
  • 문서 수정
  • 기타 (설명)

———

이슈 번호

———

주요 변경 사항

주요 파일, 로직, 컴포넌트 등을 구체적으로 적어주세요.

  • src/main/java/com/dokdok/book/dto/response/ReadingTimelinePreOpinionResponse.java: 타임라인 전용 사전의견 DTO 신규 추가 (gatheringId, meetingId 포함)
  • src/main/java/com/dokdok/book/dto/response/ReadingTimelineItem.java: preOpinion 타입을 ReadingTimelinePreOpinionResponse로 변경
  • src/main/java/com/dokdok/book/service/ReadingTimelineService.java: 타임라인 PRE_OPINION 생성 시 gatheringId, meetingId 포함하도록 수정
  • src/main/java/com/dokdok/book/dto/response/PersonalReadingTopicAnswerResponse.java: topic-answer 응답에서 gatheringId, meetingId 제거 (원복)
  • src/main/java/com/dokdok/book/service/PersonalReadingRecordService.java: topic-answer 응답 생성 로직을 기존 스펙으로 원복
  • src/main/java/com/dokdok/book/api/PersonalBookRecordApi.java: topic-answer Swagger 예시/설명 원복, timeline 설명에 식별자 제공 목적 명시

———

참고 사항

리뷰어가 알아야 할 추가 정보, 테스트 방법 등을 작성해주세요.

  • 관련 API 엔드포인트:
    • GET /api/book/{personalBookId}/records/timeline (items[].preOpinion.gatheringId, items[].preOpinion.meetingId 제공)
    • GET /api/book/{personalBookId}/records/topic-answer (식별자 미포함)
  • 삭제 연동 대상 API: DELETE /api/gatherings/{gatheringId}/meetings/{meetingId}/answers/me
  • 로컬 테스트:
    • ./gradlew test --tests com.dokdok.book.service.PersonalReadingRecordServiceTest

@U-hee U-hee changed the title Refator/add book list field [Refator] 책장 관련 API 수정 Feb 11, 2026
@U-hee U-hee requested review from a team, JWoong-01, Seoyoung-Kyung, dkqpeo, juhyunO and onuyyy and removed request for a team February 11, 2026 05:22
@U-hee U-hee self-assigned this Feb 11, 2026
@U-hee U-hee merged commit c224013 into dev Feb 13, 2026
1 check 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

2 participants