Skip to content

Conversation

@eraser502
Copy link
Contributor

Overview

  • 페이지 버전 목록 조회 API에서 각 페이지 dtoisProtected가 포함되도록 수정 구현했어요.

Related Issue

  • Closes

PR Checklist

  • [ ]

Additional Information

@eraser502 eraser502 self-assigned this Dec 10, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

요약

이 변경사항은 AppVersionPageSummaryResponse 및 관련 DTO에 새로운 boolean isProtected 필드를 추가합니다. 도메인 엔티티의 isProtected 속성을 저장소 JPQL 쿼리를 통해 DTO 계층까지 전파하는 일관된 구조적 변경입니다.


리뷰 노력 예측

🎯 2 (간단함) | ⏱️ ~10분

  • 주목할 영역:
    • AppVersionPageRepository.java의 두 JPQL 생성자 표현식이 새 필드를 포함하도록 일관되게 업데이트되었는지 확인 필요
    • AppVersionPageSummaryResponseAppVersionPageSummaryWithVersionResponse의 레코드 선언이 동기화되어 있는지 검증
    • toSummary() 메서드의 필드 매핑 순서가 생성자 서명과 일치하는지 확인

관련된 PR들

  • redotlabs/server\Deploy / Merge to Main #118: 동일한 DTO들(AppVersionPageSummaryResponse, AppVersionPageSummaryWithVersionResponse)과 JPQL 프로젝션을 대상으로 isProtected 부울 컴포넌트를 추가하는 주요 PR로, 이번 변경과 동일한 엔티티들을 다룸

🐰 작은 필드 하나, 쿼리 셋,
DTO들 춤을 추며 맞춰 가네!
보호 상태를 전하러 가는
데이터의 여정, 우아하게 🌟
구조 맞춰, 모두 행복하게!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.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
Description check ✅ Passed PR 설명이 기본 템플릿 구조는 따르고 있으나, Overview 섹션에만 간략한 설명이 있고 Related Issue와 PR Checklist는 미완성 상태입니다.
Title check ✅ Passed PR 제목이 변경 사항의 주요 목적을 명확하게 설명하고 있습니다. isProtected 필드를 페이지 DTO에 추가하는 기능 구현을 정확히 요약하고 있습니다.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/builder/cms-and-site-api

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.

@eraser502 eraser502 changed the title Feature / builder / cms and site api Feature / builder / cms and site api / Add isProtected field to page DTO Dec 10, 2025
@eraser502 eraser502 merged commit 0dbf2c8 into develop Dec 10, 2025
1 check passed
@coderabbitai coderabbitai bot mentioned this pull request Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants