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

[Fix/#194] 선호시간 선택 후 POST시 response 값에 따라 처리 수정 #201

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

eastlaw80
Copy link
Contributor

@eastlaw80 eastlaw80 commented Oct 12, 2023

✨ 해당 이슈 번호 ✨

#194

todo

  • 가능시간 post시 response로 201코드가 돌아온 경우에만 다음 페이지로 라우팅 처리
  • 그외에 400번대 오류 돌아오면 싹다 에러페이지로 라우팅
  • 에러코드 400인 경우에만 에러메세지 alert주고 해당 페이지 머무르기

📌 내가 알게 된 부분

  • 서버에서 보내주기에 따라, 400코드 에러가 try에서 잡힐 수도, catch에서 잡힐 수도 있다! 서버와 협의 필요.
  • 서버에서 리스폰스로 온 값의 에러처리를 할때, 타입스크립트 에서는 isAxiosError와 .response를 이용하여 타입스크립트의 안정성을 지킨다...!
  • AxiosResponse -> 요놈 좀 문제가 있는 개념이다. 응답 시 응답 데이터 외의 header나 그외의 것들을 사용해야 하는것 아니라면, 이걸 쓸 필요 없이 그냥 응답 데이터에만 타입 지정해주고 쓰는게 낫다고 GPT가 그랬다. 응답 데이터를 이용하다가 발생한 타입에러에 관한 트러블 슈팅을 하다가 발견했다.

📌 공유하고 싶은 부분

  • 400에러 시에만 alert로 전달받은 메세지를 띄우게 함
  • 그 외에 catch로 간 에러는 전부 에러페이지로
  • try에서도 201코드만 다음페이지 넘어가고 혹시 몰라서 다른건 전부 에러페이지로 가게 뒀다.

📌 질문할 부분

스크린샷 2023-10-12 오전 10 59 20
  • 타입 에러를 수정하려고 GPT에 물어봤더니 AxiosReponse를 지우래서 지웠더니 해결됐음.
스크린샷 2023-10-12 오전 11 00 44
  • 그래서 아래에 같은 코드를 위처럼 바꿈. 나중에 아래도 axiosresponse 지움. 근데 이거 재훈이형이 썼던데 axiosresponse를 쓴 이유가 따로 있나?

📌스크린샷

스크린샷 2023-10-12 오전 11 13 48

@eastlaw80 eastlaw80 added fix fix 동헌 동헌이의 개발 라벨 labels Oct 12, 2023
@eastlaw80 eastlaw80 self-assigned this Oct 12, 2023
@vercel
Copy link

vercel bot commented Oct 12, 2023

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

Name Status Preview Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview Oct 12, 2023 4:56am

@simeunseo simeunseo changed the title Fix/time post error/#194 선호시간 선택 후 POST시 response 값에 따라 처리 수정. [Fix/#194] 선호시간 선택 후 POST시 response 값에 따라 처리 수정 Oct 12, 2023
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.

넘 고생했습니다! 분기처리 잘 된 것 같습니다. 오류메시지로 alert 띄우는 거 좋네요 참고하겠습니다!!

@ljh0608
Copy link
Member

ljh0608 commented Oct 16, 2023

심신미약으로 그랬던 것 같습니다 죄송합니다 진행시켜주세요

@ljh0608 ljh0608 merged commit 567caab into develop Nov 3, 2023
2 checks passed
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