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

fix: 없는 페이지에 접근하려 할 경우 404로 이동 #501

Merged

nyj001012 merged 1 commit into main from 449-bug-없는-페이지-접근

Mar 13, 2024

Collaborator

nyj001012 commented Mar 13, 2024 •

edited

Loading

✅ 풀_리퀘스트 체크리스트

PR 제목: [feat/fix/refactor...] 작업 내용 한 줄 요약 (브랜치 이름) #이슈번호
commit message 가 적절한지 확인해주세요.
적절한 branch 로 요청했는지 확인해주세요.
Assignees, Label 을 붙여주세요.
주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer 로 등록해주세요.
PR이 승인된 경우 해당 브랜치는 삭제해 주세요!

🔄 변경 사항

무엇을 합니다!

errorPage.js에 404 추가
router.js에서 routeInfo에 정의되지 않은 페이지로 이동하려 할 경우, ErrorPage의 인자값을 404로 전달

📎 변경한 이유

없는 페이지에 접근할 때 401로 리다이렉트

closes: #449

👨🏻‍💻 테스트 체크리스트

e2e

📌 변경 및 주의 사항


          feat(route): 없는 페이지에 접근하려 할 경우 404로 이동

97333e1

nyj001012 added bug frontend labels

nyj001012 self-assigned this

nyj001012 linked an issue

that may be closed by this pull request

[bug] 없는 페이지 접근 #449

Closed

donghyun1998 approved these changes

View reviewed changes

frontend/src/pages/errorPage.js

Comment on lines +14 to +16

+                    case 404:
+                      comment = "그런 거 없다.";
+                      break;

Contributor

donghyun1998 Mar 13, 2024

그런거 없다.

nyj001012 merged commit 3c2d9fa into main

6 checks passed

nyj001012 deleted the 449-bug-없는-페이지-접근 branch

March 13, 2024 07:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels