Feat/140/error page

[SeokChan-Lee]

❓이슈

• close Feat: 에러 페이지 제작 #140

✍️ Description

작업 내용

• 전역 not-found, error 페이지 구현
  • not-found 페이지 버튼 = '/' 루트 페이지
  • error 페이지 버튼 = reset으로 에러 상태의 컴포넌트를 재실행 시킴
    

✅ Checklist

PR

• Branch Convention 확인

  feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

• Base Branch 확인
• 적절한 Label 지정
• Assignee 및 Reviewer 지정

Test

• 로컬 작동 확인

Additional Notes

• Error 메세지 AxiosEroor 상태 코드로 분기 처리

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
taskify	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 17, 2025 7:16am

[ToKyun02]

작업하시느라 고생하셨습니다!

전체적으로 잘 구현하셨는데, 한가지 개선하면 좋을 것 같은 점이 있어서 코멘트 남깁니다!

에러 메시지를 가공하지 않고 바로 출력하다 보니 하단 이미지 처럼 보기 안좋은? Log메시지로 보여서 어느정도 가공해야 할 듯 싶습니다!

[SeokChan-Lee]

우선 도균님이 피드백 주신 내용을 보고 유저가 보는 error 페이지에 굳이 error.message를 그대로 노출 시키는게 보기 않좋은거 같아
401,500 에러를 분기하여 수정했습니다.
다만 서버측에서 보내는 error.message에 error 상태인 401,500 이 포함되지 않는 것 같아 기본 메세지인
'예기치 않은 오류가 발생했습니다. 잠시 후 다시 시도해 주세요.' 만 출력되는 것 같아요...
이 부분 처러에 관해서는 내일 팀미팅에서 따로 다뤘으면 좋겠습니다 !

[ToKyun02]

우선 도균님이 피드백 주신 내용을 보고 유저가 보는 error 페이지에 굳이 error.message를 그대로 노출 시키는게 보기 않좋은거 같아
401,500 에러를 분기하여 수정했습니다.
다만 서버측에서 보내는 error.message에 error 상태인 401,500 이 포함되지 않는 것 같아 기본 메세지인
'예기치 않은 오류가 발생했습니다. 잠시 후 다시 시도해 주세요.' 만 출력되는 것 같아요...
이 부분 처러에 관해서는 내일 팀미팅에서 따로 다뤘으면 좋겠습니다 !

401, 500에러의 경우 테스트는 아직 안해봤지만, 제 생각으론 AxiosError타입과 Error타입을 따로처리하면 분류 가능할 것 같습니다.

axios에러 객체의 경우,
error.response.status로 HTTP 상태 코드값 확인할 수 있습니다.

[ToKyun02]

useEffect는 사용하지 않기 때문에 빼주시면 될 것 같습니다!

작업하시느라 고생하셨습니다!