Skip to content

[FIX] 휴대폰 번호 검증 어노테이션 주석처리#202

Merged
88guri merged 1 commit intodevelopfrom
fix/#201/delete-phone-num-regex
Jan 23, 2026
Merged

[FIX] 휴대폰 번호 검증 어노테이션 주석처리#202
88guri merged 1 commit intodevelopfrom
fix/#201/delete-phone-num-regex

Conversation

@88guri
Copy link
Copy Markdown
Collaborator

@88guri 88guri commented Jan 23, 2026

📌 관련 이슈

✨ 변경 사항

  • 휴대폰 번호 검증 어노테이션 주석처리했습니다.

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

  • 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
  • 로컬에서 빌드 및 실행이 정상적으로 되나요?
  • 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
  • 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

버그 수정

  • 배송 정보의 전화번호 입력 검증이 완화되어 다양한 형식의 전화번호 입력이 가능해졌습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@88guri 88guri requested a review from PBEM22 January 23, 2026 13:00
@88guri 88guri self-assigned this Jan 23, 2026
@88guri 88guri added 🚨 Fix 버그 수정 🐵 시현 시현이에요 ✋Participation 공구 참여, 관리 labels Jan 23, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

DeliveryInfo 엔티티의 phone 필드에서 휴대폰번호 정규식 검증을 비활성화했습니다. Pattern 검증 import를 제거하고 해당 annotation을 주석 처리하여 형식 검증을 일시적으로 해제했습니다.

Changes

코호트 / 파일 변경 내용
휴대폰번호 검증 비활성화
DeliveryInfo.java
Pattern import 제거 및 phone 필드의 @Pattern 검증 annotation 주석 처리 (정규식 검증 비활성화)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • PBEM22
🚥 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
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 주요 변경사항을 명확하게 반영하고 있습니다. 휴대폰 번호 검증 어노테이션 주석 처리라는 핵심 변경 내용을 정확하게 설명하고 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경이 연결 이슈 #201의 요구사항을 충족합니다. 휴대폰번호 정규식 검증을 주석 처리하여 비활성화했으며, 이는 요청된 목적과 일치합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 연결 이슈 #201의 범위 내에 있습니다. 휴대폰 번호 검증 어노테이션 처리만 진행되었으며, 다른 무관한 변경은 없습니다.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

Copy link
Copy Markdown
Collaborator

@PBEM22 PBEM22 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@88guri 88guri merged commit 118b91c into develop Jan 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 Fix 버그 수정 ✋Participation 공구 참여, 관리 🐵 시현 시현이에요

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 휴대폰번호 정규식 삭제

2 participants