Skip to content

feat: 지원서 조회 관련 응답에 이미지 URL 추가#810

Merged
whqtker merged 1 commit into
developfrom
feat/809-add-university-image-urls-to-applications-response
Jul 9, 2026
Merged

feat: 지원서 조회 관련 응답에 이미지 URL 추가#810
whqtker merged 1 commit into
developfrom
feat/809-add-university-image-urls-to-applications-response

Conversation

@whqtker

@whqtker whqtker commented Jul 9, 2026

Copy link
Copy Markdown
Member

관련 이슈

작업 내용

특이 사항

리뷰 요구사항 (선택)

@whqtker whqtker self-assigned this Jul 9, 2026
@whqtker whqtker added the 기능 label Jul 9, 2026
@coderabbitai

coderabbitai Bot commented Jul 9, 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: 1abf313c-1bf7-4ea2-9c25-f8121bd5dae8

📥 Commits

Reviewing files that changed from the base of the PR and between ac51829 and d3e1ab9.

📒 Files selected for processing (2)
  • .gitignore
  • src/main/java/com/example/solidconnection/application/dto/ApplicantsResponse.java

Walkthrough

  1. 📦 ApplicantsResponse 스키마 확장
    • 레코드에 logoImageUrl, backgroundImageUrl 필드를 새로 추가했어요.
    • of() 팩토리 메서드가 University 객체에서 이 값들을 가져와 채워주도록 확장됐답니다.
  2. 🙈 .gitignore 업데이트
    • .omc/docs/superpowers/ 경로를 Git 추적 대상에서 제외했어요.

Estimated code review effort: 1 (Trivial) | ~3분

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (3 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning 관련 이슈만 있고 작업 내용과 특이 사항이 비어 있어 설명이 거의 템플릿 상태입니다. 작업 내용과 특이 사항에 실제 변경 사항과 영향 범위를 간단히 채워 주세요.
Out of Scope Changes check ⚠️ Warning 지원서 조회와 무관한 .gitignore 변경이 함께 포함되어 있어 범위를 벗어납니다. 무관한 .gitignore 변경은 분리하거나 제거하고, 지원서 조회 응답 변경만 남겨 주세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 지원서 조회 응답에 이미지 URL을 추가하는 변경을 정확히 요약합니다.
Linked Issues check ✅ Passed ApplicantsResponse에 로고와 배경 이미지 URL을 추가해 #809의 요구를 충족합니다.
✨ 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/809-add-university-image-urls-to-applications-response

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.

@whqtker whqtker added the 최종 리뷰 최소 1명 필수 label Jul 9, 2026
@whqtker whqtker merged commit 69683f9 into develop Jul 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

기능 최종 리뷰 최소 1명 필수

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 지원서 조회 응답에 로고, 대학 배경 URL을 추가한다.

2 participants