Skip to content

[Refactor] 온보딩 OCR에서 국가 반환 형태 LocalizedItemResponse로 수정#236

Merged
jeong1112 merged 2 commits intodevelopfrom
refactor/#235-onboard-ocr
Apr 10, 2026
Merged

[Refactor] 온보딩 OCR에서 국가 반환 형태 LocalizedItemResponse로 수정#236
jeong1112 merged 2 commits intodevelopfrom
refactor/#235-onboard-ocr

Conversation

@jeong1112
Copy link
Copy Markdown
Collaborator

@jeong1112 jeong1112 commented Apr 10, 2026

Related issue 🛠

Summary by CodeRabbit

릴리스 노트

  • 개선 사항
    • 여권 OCR 인식 결과에서 국가 정보가 국가 코드와 국가명을 함께 제공하도록 개선되었습니다.
    • 멤버 완성도 조회 시 비자 관련 불필요한 검증 로직이 제거되어 시스템 안정성이 개선되었습니다.

@jeong1112 jeong1112 self-assigned this Apr 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b8bf8d29-82d0-46bf-a9a0-87e78607d7e1

📥 Commits

Reviewing files that changed from the base of the PR and between 797422f and 1046908.

📒 Files selected for processing (3)
  • src/main/java/org/sopt/kareer/domain/member/dto/response/OcrPassportResponse.java
  • src/main/java/org/sopt/kareer/domain/member/entity/enums/Country.java
  • src/main/java/org/sopt/kareer/domain/member/service/MemberService.java

📝 Walkthrough

워크스루

온보딩 OCR 과정의 국적 반환 형식을 개선합니다. Country 열거형에 getCode() 메서드를 추가하고, OcrPassportResponse의 country 필드를 Country 열거형에서 LocalizedItemResponse로 변경하며, MemberService에서 비자 관련 의존성을 제거합니다.

변경사항

코호트 / 파일 요약
Country 열거형 확장
src/main/java/org/sopt/kareer/domain/member/entity/enums/Country.java
열거형 상수명을 소문자로 변환하고 밑줄을 하이픈으로 바꾸는 getCode() 메서드 추가
OCR 응답 데이터 구조 변경
src/main/java/org/sopt/kareer/domain/member/dto/response/OcrPassportResponse.java
country 필드를 Country 열거형에서 LocalizedItemResponse(code, label)로 변경하고, 팩토리 메서드를 업데이트하여 새로운 구조로 매핑
MemberService 간소화
src/main/java/org/sopt/kareer/domain/member/service/MemberService.java
getCompletion() 메서드에서 비자 저장소 조회 및 예외 처리 제거

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~12분

🐰 국적 정보를 옷 입혀주었네요,
코드와 라벨로 멋지게 꾸며서,
비자 무게도 덜어내고,
더 간결한 서비스가 태어났어요! 🌟

✨ 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/#235-onboard-ocr

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.

@jeong1112 jeong1112 merged commit 482e5ae into develop Apr 10, 2026
1 of 2 checks passed
@jeong1112 jeong1112 deleted the refactor/#235-onboard-ocr branch April 10, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] 온보딩 OCR 결과 반환값 수정

1 participant