feat/#589 프로젝트 GitHub URL 및 4기 카테고리 추가#590
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
Walkthrough프로젝트에 GitHub URL 저장 및 상세 응답 전달을 추가했습니다. 또한 ChangesGitHub URL 전달
SEMESTER_4 지원
Estimated code review effort: 2 (Simple) | ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Test Results413 tests 413 ✅ 13s ⏱️ Results for commit e2b8c78. |
📊테스트 커버리지
|
kangkings
left a comment
There was a problem hiding this comment.
고생하셨습니다! 간단한 코멘트 남겨뒀습니다!
| @@ -41,6 +42,7 @@ public static ProjectDetailResponse toResponse(Project project, | |||
| .badges(project.getBadges()) | |||
| .description(project.getDescription()) | |||
| .serviceUrl(project.getServiceUrl()) | |||
There was a problem hiding this comment.
의도하신 대로라면 두 url의 동시 존재 여부를 서비스 로직에서 검증하는게 안전해 보입니다! 다만 메인 로직이나 도메인은 아니라서 당장은 이정도도 충분해보이는데 검증 로직이 없다면 주석으로라도 현재 정책을 남겨두면 추후 개선이나 파악할 때 좋을 것 같습니다!
작업 내용
githubUrl을 추가하고 상세 조회 응답에서serviceUrl과 함께 반환합니다.Project.Category에SEMESTER_4를 추가합니다.project.github_url VARCHAR(2083) NULL컬럼을 추가하는 Flyway V34 마이그레이션을 포함합니다.변경 이유
운영 서비스 URL과 GitHub URL은 서로 독립적인 값입니다. 두 URL이 모두 있거나 하나만 있을 수 있으며, 모두 없으면 연결할 링크가 없는 프로젝트입니다.
isActive나linkType같은 중복 상태값은 추가하지 않았습니다.테스트
JAVA_HOME=$(/usr/libexec/java_home -v 21) ./gradlew test --tests org.ject.support.domain.project.service.ProjectServiceTest -x jacocoTestCoverageVerification성공./gradlew test: 로컬 Docker 미실행으로 Redis Testcontainers 초기화 관련 32건 실패배포 및 데이터 반영
관련 이슈
#589
Summary by CodeRabbit
새로운 기능
문서
데이터베이스