Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

강원대 FE_허윤수 4주차 과제 STEP3 #53

Open
wants to merge 56 commits into
base: sugoring
Choose a base branch
from

Conversation

sugoring
Copy link

@sugoring sugoring commented Jul 19, 2024

멘토님 안녕하세요!

이번 PR에서는 결제 페이지의 폼을 React Hook Form을 사용하여 리팩터링하고, 유효성 검사 로직을 개선했습니다. 피드백을 위해 시간을 내주셔서 감사합니다!


3단계 - React Hook Form 리팩터링

  • 기존 폼을 React Hook Form으로 변경
  • 폼 필드와 유효성 검사를 관리

yuni added 30 commits July 17, 2024 05:02
- 상세 요구사항은 각 단계에서 정리 할 예정
- API 응답 데이터 구조 변경에 따른 상품 상세 정보 접근 방식 수정 (productDetail.detail)
- 상품 및 옵션 정보 로딩 상태 및 에러 처리 로직 추가
- React Query 활용 방식 개선: 데이터 fetching 함수 분리 및 경로 함수 추가
- enabled 옵션 추가를 통해 productId 값 유효성 검사 후 쿼리 실행하여 불필요한 API 호출 방지
yuni added 26 commits July 17, 2024 21:46
- '@/components/common/Button'에서 가져온 커스텀 버튼 컴포넌트를 사용하여 결제 버튼 대체
- '@/components/common/Form/Input/UnderlineTextField'에서 가져온 커스텀 입력 필드 컴포넌트를 사용하여 영수증 번호 입력 필드 대체
- 결제 페이지 레이아웃을 2열 구조로 수정하여 선물 메시지 및 선물 내역을 좌측에, 결제 정보를 우측에 배치
- 각 섹션에 borderWidth, padding, borderRadius, background, boxShadow 스타일 적용
- 카드 메시지를 입력하지 않은 경우 경고 메시지 표시
- 카드 메시지가 100자를 초과할 경우 경고 메시지 표시
- 현금영수증 체크 시 번호 입력 필드 표시
- 번호가 입력되지 않은 경우 경고 메시지 표시
- 현금영수증 번호가 숫자인지 유효성 검사 추가
- React Hook Form 설치 및 기본 설정
- 기존 상태 관리 및 유효성 검사를 React Hook Form으로 전환
- 현금영수증 체크 시 번호 입력 필드 표시
- 번호가 입력되지 않은 경우 경고 메시지 표시
- 현금영수증 번호가 숫자인지 유효성 검사 추가
- 카드 메시지를 입력하지 않은 경우 경고 메시지 표시
- 카드 메시지가 100자를 초과할 경우 경고 메시지 표시
Copy link

@danmin20 danmin20 left a comment

Choose a reason for hiding this comment

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

이전 스텝과 비교했을 때 추가적으로 리뷰드릴 내용은 없는 것 같습니다~!

@danmin20
Copy link

컨플릭트 해결 후 머지해주시면 될 것 같아요~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants