Skip to content

refactor: 미사용 컬럼 대신 extra_info 사용#743

Merged
whqtker merged 1 commit into
developfrom
refactor/742-use-extra-info-for-optional-fields
Jun 9, 2026
Merged

refactor: 미사용 컬럼 대신 extra_info 사용#743
whqtker merged 1 commit into
developfrom
refactor/742-use-extra-info-for-optional-fields

Conversation

@whqtker

@whqtker whqtker commented Jun 9, 2026

Copy link
Copy Markdown
Member

관련 이슈

작업 내용

tuition_fee_type, details_for_apply, details_for_major, details_for_english_course, details 컬럼은 JSON 타입의 extra_info 컬럼에서 유지합니다.

이에 응답 DTO에서 extra_info 필드 사용하도록 변경합니다.

#741 실제 컬럼, 필드는 모의지원 후 제거할 예정입니다. (혹시 모를 상황 대비)

특이 사항

리뷰 요구사항 (선택)

@whqtker whqtker self-assigned this Jun 9, 2026
@whqtker whqtker added 리팩터링 최종 리뷰 최소 1명 필수 labels Jun 9, 2026
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

🔍 Walkthrough

이번 변경사항은 대학 지원 정보 응답 DTO를 재정의하는 작업입니다:

  1. import 추가

    • Map 타입 사용을 위해 java.util.Map import가 추가됨
  2. 레코드 필드 구성 변경

    • 기존 필드 제거: tuitionFeeType, detailsForApply, detailsForMajor, detailsForEnglishCourse, details
    • 신규 필드 추가: Map<String, String> 타입의 extraInfo
    • 기존 필드 유지: accommodationUrl, englishCourseUrl 등 URL/요건 관련 필드들이 재정렬됨
  3. 팩토리 메서드 인자 조정

    • univApplyInfo.getTuitionFeeType().getKoreanName() 매핑 제거
    • univApplyInfo.getExtraInfo() 매핑 추가
    • 레코드 생성자 호출 시 인자 목록이 새로운 필드 구성에 맞춰 정렬됨

🎯 Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes


👥 Suggested reviewers

  • sukangpunch
  • Gyuhyeok99
  • wibaek
  • Hexeong
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 반영하고 있으며, 미사용 컬럼 제거 및 extra_info 필드 사용이라는 주요 리팩토링 목표를 간결하게 표현했습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 #742의 목표인 'extra_info 필드를 사용하도록 리팩토링'에 부합합니다. UnivApplyInfoDetailResponse 레코드가 개별 필드 제거 후 extra_info Map을 추가하여 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed 변경사항이 모두 UnivApplyInfoDetailResponse 레코드 리팩토링에 집중되어 있으며, #742의 범위 내에서 extra_info 필드 도입과 관련 필드 제거만 수행했습니다.
Description check ✅ Passed PR 설명에서 관련 이슈, 작업 내용, 특이 사항 섹션이 모두 포함되었으나, 작업 내용이 간략하고 리뷰 요구사항이 미기재된 상태입니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 refactor/742-use-extra-info-for-optional-fields

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5aeca71bc0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@sukangpunch sukangpunch 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.

고생하셨습니다!

@whqtker whqtker merged commit 850676c into develop Jun 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.

refactor: 대학 검색 응답에서 extra_info 필드 사용하도록

2 participants