Skip to content

[Feat/#309]에러 처리와 약관 동의 #313

Merged
seohyun06 merged 4 commits intodevelopfrom
feat/309-error-boundary
Oct 29, 2025
Merged

[Feat/#309]에러 처리와 약관 동의 #313
seohyun06 merged 4 commits intodevelopfrom
feat/309-error-boundary

Conversation

@seohyun06
Copy link
Contributor

@seohyun06 seohyun06 commented Oct 22, 2025

📖 스토리북 링크 (Storybook Link)

변경된 UI 컴포넌트를 시각적으로 확인하고 인터랙션해볼 수 있는 스토리북 링크를 첨부합니다.


🔗 관련 이슈 번호 (Related Issue Number)

이 Pull Request가 해결하고자 하는 이슈의 번호를 기재합니다. 이슈 연결을 통해 작업의 배경과 목적을 쉽게 파악할 수 있습니다. (예: Closes #123)

  • Closes #

✨ PR 세부 내용 (PR Details)

이번 PR에서 어떤 작업을 했는지 상세하게 설명합니다. 코드 변경의 이유, 구현 방식, 주요 로직 등을 중심으로 작성하면 리뷰어가 이해하는 데 도움이 됩니다.

작업 개요

  • 400번대 에러 나면 404 에러 페이지로 이동
  • 500번대 에러 나면 네트워크 오류 페이지로 이동
    주요 변경사항

기술적 구현 사항


📸 참고할 스크린샷/영상 (Screenshots/Videos)

작업 내용과 관련된 시각적인 자료가 있다면 첨부합니다. UI 변경이나 새로운 기능의 동작 방식을 보여주는 스크린샷 또는 영상을 활용하면 리뷰어의 이해를 돕습니다.

UI 변경 전/후 스크린샷
image
image
image
image

기능 동작 영상

추가 참고 자료

@seohyun06 seohyun06 self-assigned this Oct 22, 2025
Copy link
Contributor

@00kang 00kang left a comment

Choose a reason for hiding this comment

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

에러 상황 테스트 해보셨을까요?
UI도 첨부해주시면 좋을 것 같습니다!

@seohyun06
Copy link
Contributor Author

에러 상황 테스트 해봤습니다! UI도 첨부했습니다 확인해주세요

@00kang
Copy link
Contributor

00kang commented Oct 27, 2025

피그마 디자인을 따라 회색배경에 에러 컨텐츠는 가운데 정렬되는 게 좋을 것 같습니다.

image

Copy link
Contributor

@00kang 00kang left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@seohyun06 seohyun06 merged commit 9909729 into develop Oct 29, 2025
1 check passed
@seohyun06 seohyun06 deleted the feat/309-error-boundary branch October 29, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants