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 #55

Open
wants to merge 18 commits into
base: yunn23
Choose a base branch
from

Conversation

yunn23
Copy link

@yunn23 yunn23 commented Jul 19, 2024

4주차 과제 STEP3 제출합니다.

기존의 input/form을 react-hook-form을 사용하여 재구성하였습니다.

질문 : [카드 메세지를 작성하고 현금 영수증을 체크하지 않은 경우]와 [카드메세지를 작성하고 현금영수증을 체크하고 휴대폰 번호를 적은 경우] 위 두 상태에 대해 주문이 접수되었다는 alert()를 띄우려고 onSubmit() 내부에 조건문을 작성하였는데, 처음에 1번 경우로 제출한 경우에는 제대로 제출되지 않는 것 같습니다. (주문시 콘솔이 찍히지 않고, alert()도 안 뜸) 2번으로 한 번 제출한 후에는 1번 경우에도 alert()가 뜨는데, 처음에 1번 경우가 alert()가 뜨지 않는 이유가 궁금합니다..! validation 자체는 잘 되는것으로 확인됩니다.

@yunn23 yunn23 changed the title 전남대 FE_정서윤 4주차 과제 STEP 3, 4 전남대 FE_정서윤 4주차 과제 STEP 3 Jul 19, 2024
@yunn23 yunn23 changed the title 전남대 FE_정서윤 4주차 과제 STEP 3 전남대 FE_정서윤 4주차 과제 STEP3 Jul 19, 2024
@taehwanno taehwanno self-requested a review July 19, 2024 14:52
Copy link

@taehwanno taehwanno left a comment

Choose a reason for hiding this comment

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

피드백 드려요! 새로운 라이브러리 사용시 공식문서를 전반적으로 숙지하고 활용하는게 좋겠습니다. 한번에 모든 걸 이해하기는 어렵겠지만 라이브러리에서 사용하시는 API에 한해서는 찬찬히 읽어보는게 더 느린 것처럼 보여도 시간적으로 보면 오히려 더 이득인 선택일 때가 많더라구요.

src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
src/pages/Order/index.tsx Outdated Show resolved Hide resolved
@yunn23
Copy link
Author

yunn23 commented Jul 20, 2024

step3 피드백 확인 후 모두 반영하여 수정하였습니다.
매주 과제가 나오다보니 시간에 쫓겨 공식문서를 읽어보지는 않았는데, 앞으로는 공식문서를 읽는 습관을 들이도록 노력하겠습니다!

@yunn23 yunn23 requested a review from taehwanno July 20, 2024 12:57
Copy link

@taehwanno taehwanno left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

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