Skip to content

Conversation

@zyovn
Copy link
Member

@zyovn zyovn commented Jan 5, 2026

🔗 연관된 이슈

🚀 변경 유형

  • ✨ 기능 추가 (feature)
  • 🐛 버그 수정 (fix)
  • 📝 문서 변경 (docs)
  • ♻️ 리팩토링 (refactor)
  • 🧪 테스트 추가 / 수정 (test)
  • ⚙️ 설정 변경 (chore)

📝 작업 내용

  • 앨범 생성 API DTO 수정 (생성일시, 처음 생성 여부 추가)

📸 스크린샷

처음 생성 시

image

그 이후

image

💬 리뷰 요구사항

📜 리뷰 규칙

Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.

  • P1: 꼭 반영해주세요 (Comment)
  • P2: 적극적으로 고려해주세요 (Comment)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

Summary by CodeRabbit

  • 신규 기능
    • 앨범 생성 응답에 생성 시간(createdAt) 정보가 추가되었습니다.
    • 해당 앨범이 사용자의 첫 앨범인지를 나타내는 정보(isFirst)가 응답에 포함됩니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@zyovn zyovn self-assigned this Jan 5, 2026
@zyovn zyovn added the ✨feature New feature or request label Jan 5, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 5, 2026

Walkthrough

앨범 생성 시 해당 제작자의 첫 번째 앨범 여부를 판단하는 로직을 추가했습니다. 저장소에 existsByMakerId 메서드를 추가하고, 서비스에서 이 값을 확인한 후 매퍼를 통해 응답 DTO에 createdAtisFirst 필드를 포함시킵니다.

Changes

Cohort / File(s) 변경 사항
저장소 메서드 추가
src/main/java/com/cheeeese/album/infrastructure/persistence/AlbumRepository.java
제작자 ID로 앨범 존재 여부를 확인하는 existsByMakerId(Long makerId) 메서드 추가
서비스 로직 업데이트
src/main/java/com/cheeeese/album/application/AlbumService.java
existsByMakerId를 통해 첫 번째 앨범 여부를 판단하는 isFirst 불린값 계산 로직 추가, 매퍼 호출 시 isFirst 인자 전달
매퍼 메서드 시그니처 변경
src/main/java/com/cheeeese/album/infrastructure/mapper/AlbumMapper.java
toCreationResponse(Album album)toCreationResponse(Album album, boolean isFirst) 시그니처 변경, createdAtisFirst 필드를 빌더에 추가
응답 DTO 필드 확장
src/main/java/com/cheeeese/album/dto/response/AlbumCreationResponse.java
LocalDateTime createdAt, boolean isFirst 필드 추가, LocalDateTime import 추가, 스키마 메타데이터 업데이트

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

Suggested reviewers

  • dahyun24

Poem

🐰 첫 앨범을 짚고 서는 그 순간,
제작자의 특별함을 담아내고,
시간도 함께 기록하며,
매퍼는 데이터를 정성껏 포장하네!
저장소에서 확인하고 응답에 담고,
모두가 행복한 변경이로다! 🎵

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 앨범 생성 API DTO 수정이라는 주요 변경사항을 명확하게 요약하고 있으며, 변경 내용과 완벽하게 일치합니다.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings

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.

Copy link
Contributor

@dahyun24 dahyun24 left a comment

Choose a reason for hiding this comment

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

깔끔하군!
수고하셨습니다~~

@zyovn zyovn merged commit acf198c into develop Jan 6, 2026
1 check passed
@zyovn zyovn deleted the CEEZ-33-album-dto branch January 6, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants