Skip to content

[DEV] 404 페이지 작업#140

Merged
drddyn merged 7 commits into
developfrom
dev/404page
May 25, 2026
Merged

[DEV] 404 페이지 작업#140
drddyn merged 7 commits into
developfrom
dev/404page

Conversation

@drddyn

@drddyn drddyn commented May 25, 2026

Copy link
Copy Markdown
Owner

🔗 관련 이슈

close #139

✨ 요약

404 페이지를 작업하고 연결합니다

📝 작업 내용

image

@vercel

vercel Bot commented May 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mine Ready Ready Preview, Comment May 25, 2026 4:16pm

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request implements a global 404 error handling system using a Zustand store, an axios interceptor, and a new NotFoundPage component. Key feedback includes the necessity of resetting the error state when the user navigates away from the error page to avoid persistent 404 views. The reviewer also recommended separating UI and API integration into distinct pull requests per project rules and suggested verifying the 404 page design against Figma specifications, specifically regarding the absence of a 'return to home' button.

Comment thread Mine/src/layout/Layout.tsx Outdated
Comment thread Mine/src/api/axios.ts
Comment thread Mine/src/pages/NotFoundPage.tsx
Comment thread Mine/src/stores/error.ts
@drddyn drddyn self-assigned this May 25, 2026
@drddyn drddyn merged commit 232eb16 into develop May 25, 2026
3 checks passed
@drddyn drddyn deleted the dev/404page branch May 25, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DEV] 404 페이지 작업 및 API 연결

1 participant