Skip to content

[ Fix/#192] host로그인 시 가능시간 입력안함, 비밀번호 유효성 검증에 대한 에러 해결 #199

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

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

ljh0608
Copy link
Member

@ljh0608 ljh0608 commented Oct 11, 2023

✨ 해당 이슈 번호 ✨

#192

todo

  • 403에러 (가능시간 입력 안했을 때 발생하는 에러) 가능시간 입력 모달을 띄우는 로직으로 처리 완료
  • 400에러 (비밀번호에 숫자외에 값이 들어간 에러) 로그인 실패 모달을 띄우는 로직으로 처리 완료

📌 내가 알게 된 부분

  • try catch 로직에서 catch(err) 부분에 err 는 타입이 unknown이다.
  • if(axios.isAxiosError(err)) 로직으로 감싸 Axios에러가 맞는지 확인하거나
  • if(err.response) 와 같은 타입가드로 타입을 보호? 할 수 있다. (옵셔널체이닝은 에러가 발생함)

📌 공유하고 싶은 부분

📌 질문할 부분

📌스크린샷

@ljh0608 ljh0608 added fix fix 재훈 재훈이의 개발 라벨 labels Oct 11, 2023
@ljh0608 ljh0608 self-assigned this Oct 11, 2023
@vercel
Copy link

vercel bot commented Oct 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview Oct 11, 2023 7:15am

@ljh0608 ljh0608 changed the title Fix/403error/#192 [ Fix / #192 ] host로그인 시 가능시간 입력안함, 비밀번호 유효성 검증에 대한 에러 해결 Oct 11, 2023
Copy link
Contributor

@eastlaw80 eastlaw80 left a comment

Choose a reason for hiding this comment

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

좋습니다! 에러처리한 부분 보고서 제 부분 POST 에러처리도 철저히 해놓도록 하겠습니다. 고생하셨습니다.

Copy link
Member

@simeunseo simeunseo left a comment

Choose a reason for hiding this comment

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

타입 가드..!! isAxiosError 배워갑니다 짱짱

@simeunseo simeunseo changed the title [ Fix / #192 ] host로그인 시 가능시간 입력안함, 비밀번호 유효성 검증에 대한 에러 해결 [ Fix/#192] host로그인 시 가능시간 입력안함, 비밀번호 유효성 검증에 대한 에러 해결 Oct 16, 2023
@ljh0608 ljh0608 merged commit aca78e4 into develop Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix size/M 재훈 재훈이의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants