Skip to content

feat/#589 프로젝트 GitHub URL 및 4기 카테고리 추가#590

Merged
023-dev merged 1 commit into
devfrom
feat/project-links-semester4
Jul 14, 2026
Merged

feat/#589 프로젝트 GitHub URL 및 4기 카테고리 추가#590
023-dev merged 1 commit into
devfrom
feat/project-links-semester4

Conversation

@023-dev

@023-dev 023-dev commented Jul 14, 2026

Copy link
Copy Markdown
Member

작업 내용

  • 프로젝트에 nullable githubUrl을 추가하고 상세 조회 응답에서 serviceUrl과 함께 반환합니다.
  • Project.CategorySEMESTER_4를 추가합니다.
  • project.github_url VARCHAR(2083) NULL 컬럼을 추가하는 Flyway V34 마이그레이션을 포함합니다.
  • 프로젝트 상세 응답 및 4기 요약 동작 테스트를 보강합니다.

변경 이유

운영 서비스 URL과 GitHub URL은 서로 독립적인 값입니다. 두 URL이 모두 있거나 하나만 있을 수 있으며, 모두 없으면 연결할 링크가 없는 프로젝트입니다. isActivelinkType 같은 중복 상태값은 추가하지 않았습니다.

테스트

  • 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건 실패

배포 및 데이터 반영

  • 스키마 변경: Flyway V34 적용 필요
  • 운영 S3에 4기 프로젝트 이미지 88개 업로드 및 CloudFront 응답 확인 완료
  • 운영 DB의 4기 프로젝트 등록과 기존 7개 프로젝트 URL 변경은 별도 운영 데이터 작업으로 진행합니다.

관련 이슈

#589

Summary by CodeRabbit

  • 새로운 기능

    • 프로젝트 상세 정보에 GitHub 저장소 URL이 표시됩니다.
    • 프로젝트 분류에 4기(SEMESTER_4)가 추가되었습니다.
  • 문서

    • Jectalk 조회 API 문서에서 SEMESTER_4 선택 옵션이 안내됩니다.
  • 데이터베이스

    • 프로젝트의 GitHub URL 저장을 지원합니다.

@023-dev 023-dev added the enhancement New feature or improvement label Jul 14, 2026
@023-dev 023-dev self-assigned this Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 04d6dea6-b6a7-4445-81ab-f278b0a87cc0

📥 Commits

Reviewing files that changed from the base of the PR and between 610c310 and e2b8c78.

📒 Files selected for processing (5)
  • src/main/java/org/ject/support/domain/jectalk/controller/JectalkApiSpec.java
  • src/main/java/org/ject/support/domain/project/dto/ProjectDetailResponse.java
  • src/main/java/org/ject/support/domain/project/entity/Project.java
  • src/main/resources/db/migration/V34__add_github_url_to_project.sql
  • src/test/java/org/ject/support/domain/project/service/ProjectServiceTest.java

Walkthrough

프로젝트에 GitHub URL 저장 및 상세 응답 전달을 추가했습니다. 또한 SEMESTER_4 카테고리를 도입하고 API 문서와 프로젝트 요약 조회 테스트를 갱신했습니다.

Changes

GitHub URL 전달

Layer / File(s) Summary
GitHub URL 저장 및 상세 응답
src/main/java/org/ject/support/domain/project/entity/Project.java, src/main/resources/db/migration/V34__add_github_url_to_project.sql, src/main/java/org/ject/support/domain/project/dto/ProjectDetailResponse.java, src/test/java/org/ject/support/domain/project/service/ProjectServiceTest.java
Projectproject 테이블에 githubUrl을 추가하고, 프로젝트 상세 응답에 값을 매핑하며 관련 테스트를 확장했습니다.

SEMESTER_4 지원

Layer / File(s) Summary
SEMESTER_4 카테고리 및 문서
src/main/java/org/ject/support/domain/project/entity/Project.java, src/main/java/org/ject/support/domain/jectalk/controller/JectalkApiSpec.java, src/test/java/org/ject/support/domain/project/service/ProjectServiceTest.java
SEMESTER_4 열거형 값을 추가하고 API 파라미터 설명과 카테고리 요약 검증에 반영했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

깡충 뛰는 토끼가 URL을 심고
네 번째 기수를 살짝 불렀네.
응답에는 GitHub 길이 열리고
문서와 테스트도 함께 웃네.
당근처럼 깔끔한 변경이라네!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 프로젝트 GitHub URL 추가와 4기 카테고리 추가라는 주요 변경을 잘 요약한 제목입니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/project-links-semester4

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.

@github-actions

Copy link
Copy Markdown

Test Results

413 tests   413 ✅  13s ⏱️
 77 suites    0 💤
 77 files      0 ❌

Results for commit e2b8c78.

@github-actions

Copy link
Copy Markdown

📊테스트 커버리지

Overall Project 85.16% 🍏
Files changed 100% 🍏

File Coverage
Project.java 100% 🍏

@023-dev
023-dev requested a review from kangkings July 14, 2026 11:00

@kangkings kangkings left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

고생하셨습니다! 간단한 코멘트 남겨뒀습니다!

@@ -41,6 +42,7 @@ public static ProjectDetailResponse toResponse(Project project,
.badges(project.getBadges())
.description(project.getDescription())
.serviceUrl(project.getServiceUrl())

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

의도하신 대로라면 두 url의 동시 존재 여부를 서비스 로직에서 검증하는게 안전해 보입니다! 다만 메인 로직이나 도메인은 아니라서 당장은 이정도도 충분해보이는데 검증 로직이 없다면 주석으로라도 현재 정책을 남겨두면 추후 개선이나 파악할 때 좋을 것 같습니다!

@023-dev
023-dev merged commit d18a7e2 into dev Jul 14, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants