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.

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

[ Fix ] 에러페이지 홈으로 클릭시 이동 변경사항 반영 #352

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

yarimu
Copy link
Contributor

@yarimu yarimu commented Nov 11, 2024

#️⃣ Related Issue

Closes #321

✅ Done Task

  • 홈으로 클릭시 토큰, 롤 유무 롤에따라 이동페이지 변경

💎 PR Point

1️⃣ 토큰이랑 롤 둘다 있는데 에러

  • 선배일 경우 -> promiseList
  • 후배일 경우 -> juniorPromise

2️⃣ 토큰 또는 롤 없어서 에러났다
-> 로그인페이지로(기존과 동일: 로컬스토리지 비우고 루트로 이동)

📸 Screenshot

juniorPromise api 망가뜨리고 후배계정으로 테스트 해본 거라 임시방편으로 영상찍을때는 이동주소 promiseList로 해놓고 테스트 한 영상입니다! 코드는 다시 juniorPromise로 잘 되어있습니다!

  • 토큰, 롤 둘 다 있는 경우
default.mov
  • 토큰 없는 경우
default.mov
  • 롤 없는 경우
2024-11-11.11.20.40.mov

@yarimu yarimu added the 🛠 Fix 기존의 버그 수정 label Nov 11, 2024
@yarimu yarimu requested review from j-nary and se0jinYoon November 11, 2024 14:41
@yarimu yarimu self-assigned this Nov 11, 2024
Copy link
Collaborator

@se0jinYoon se0jinYoon left a comment

Choose a reason for hiding this comment

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

lggggggtm 완굿굿굿

@yarimu yarimu changed the title [ Fix ] 에러페이지 홈으로 클릭시 이동 변경사항 반영 [ Fix ] 에러페이지 홈으로 클릭시 이동 변경사항 반영 Nov 11, 2024
@yarimu yarimu added the 예림 label Nov 11, 2024
Copy link
Member

@j-nary j-nary left a comment

Choose a reason for hiding this comment

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

LGTM

@yarimu yarimu merged commit 9cf5e15 into develop Nov 17, 2024
1 check passed
@yarimu yarimu deleted the feat/#321/newErrorPage branch November 17, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/s 예림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Feat ] 에러페이지 바뀐 버전으로 변경
3 participants