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.

Sign up for GitHub

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

Jump to bottom

🎉 전역적 에러 처리 추가 #124

Merged
merged 2 commits into from
Nov 28, 2023
Merged

🎉 전역적 에러 처리 추가 #124

merged 2 commits into from
Nov 28, 2023

Conversation

oaoong
Copy link
Collaborator

@oaoong oaoong commented Nov 27, 2023

- 목적

관련 티켓 번호: 316


- 주요 변경 사항

  • 라우트 별 에러 처리 추가
  • 공통 에러 처리 추가

기타 사항 (선택)

  • suggestion/id 라우트에 본인 것이 아닌 경우 접근 제한 추가
  • throwing 된 에러에 맞는 공통 에러 페이지 추가

- 스크린샷 (선택)

@oaoong oaoong added the D-0 label Nov 27, 2023
@oaoong oaoong self-assigned this Nov 27, 2023
@oaoong oaoong changed the title Nabi 316 🎉 전역적 에러 처리 추가 Nov 27, 2023
juyeon-park
juyeon-park approved these changes Nov 27, 2023
View reviewed changes
Copy link
Contributor

@juyeon-park juyeon-park left a comment

Choose a reason for hiding this comment

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

👍

src/app/(root)/(routes)/chatrooms/[chatRoomId]/page.tsx Show resolved Hide resolved
src/app/error.tsx Show resolved Hide resolved
src/app/not-found.tsx Show resolved Hide resolved
@oaoong oaoong merged commit 51b4b39 into develop Nov 28, 2023
4 checks passed
@oaoong oaoong deleted the NABI-316 branch November 28, 2023 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juyeon-park juyeon-park juyeon-park approved these changes

@doggopawer doggopawer Awaiting requested review from doggopawer

Assignees

@oaoong oaoong

Labels
D-0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oaoong @juyeon-park