Skip to content

refactor: university_info_for_apply 미사용 컬럼 제거#775

Merged
whqtker merged 1 commit into
developfrom
refactor/741-remove-unused-columns-from-university-info-for-apply
Jun 19, 2026
Merged

refactor: university_info_for_apply 미사용 컬럼 제거#775
whqtker merged 1 commit into
developfrom
refactor/741-remove-unused-columns-from-university-info-for-apply

Conversation

@whqtker

@whqtker whqtker commented Jun 19, 2026

Copy link
Copy Markdown
Member

관련 이슈

작업 내용

선택 컬럼을 extraInfo로 이관함에 따른 미사용 컬럼 제거합니다.

  • stage db 미사용 컬럼에 값이 없음을 확인한다.
  • prod db 미사용 컬럼에 값이 없음을 확인한다.

특이 사항

리뷰 요구사항 (선택)

- UnivApplyInfo 엔티티에서 tuitionFeeType, detailsForApply, detailsForMajor,
  detailsForEnglishCourse, details 필드 제거
- TuitionFeeType 열거형 삭제
- UnivApplyInfoColumn에서 대응 enum 상수 5개 제거
- AdminUnivApplyInfoRowSaver의 ImportData 및 switch case 정리
- data.sql INSERT 구문에서 제거된 컬럼 반영
- V54 마이그레이션으로 DB 컬럼 DROP
@coderabbitai

coderabbitai Bot commented Jun 19, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

  1. 도메인 정리

    • UnivApplyInfo 엔터티에서 tuitionFeeType, detailsForApply, detailsForMajor, detailsForEnglishCourse, details 5개 필드를 제거하고, detailsForAccommodationextraInfo 앞에 위치하는 구조로 변경됩니다.
    • TuitionFeeType enum 클래스 전체가 삭제됩니다.
    • UnivApplyInfoColumn에서 TUITION_FEE_TYPE, DETAILS_FOR_APPLY, DETAILS_FOR_MAJOR, DETAILS_FOR_ENGLISH_COURSE, DETAILS 상수가 제거됩니다.
  2. DB 마이그레이션 추가

    • V54__remove_unused_columns_from_univ_apply_info.sql이 추가되어, university_info_for_apply 테이블에서 위 5개 컬럼을 순차적으로 DROP합니다.
  3. 어드민 임포트 서비스 동기화

    • AdminUnivApplyInfoRowSaver에서 TuitionFeeType import가 제거되고, UnivApplyInfo 생성자 호출 인자와 ImportData 내부 필드가 새 구조에 맞게 갱신됩니다.
    • switch 케이스에서 삭제된 필드 분기가 제거되고, detailsForAccommodation 케이스가 추가됩니다.
  4. 시드 데이터 및 테스트 픽스처 갱신

    • data.sql의 INSERT 컬럼 구성이 semester_available_for_dispatch, details_for_language, details_for_accommodation 중심으로 교체되어 18개 대학 행이 재정의됩니다.
    • UnivApplyInfoFixtureBuilder에서 HOME_UNIVERSITY_PAYMENT 인자 및 정적 import가 제거되고 생성자 인자가 재구성됩니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • lsy1307
  • Hexeong
  • JAEHEE25
  • sukangpunch
  • Gyuhyeok99
🚥 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 제목이 변경사항의 핵심을 정확하게 반영하고 있습니다. 미사용 컬럼 제거라는 주요 작업이 명확하게 표현되어 있습니다.
Linked Issues check ✅ Passed PR의 모든 코드 변경사항이 #741 이슈의 목표를 충족합니다. 다섯 개의 미사용 컬럼 제거, TuitionFeeType 열거형 삭제, 데이터베이스 마이그레이션 포함.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #741 이슈의 범위 내에 있습니다. 미사용 컬럼 제거 관련 코드 수정과 데이터베이스 마이그레이션만 포함되어 있습니다.
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/741-remove-unused-columns-from-university-info-for-apply

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: e03c4ecbba

ℹ️ 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".

@whqtker whqtker merged commit 4622ace into develop Jun 19, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: university_info_for_apply 미사용 컬럼 제거

1 participant