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

my-page로 임의 접근 시 로그인 alert가 3번 발생하는 issue 해결 #186

Closed
jinyoung234 opened this issue Jun 13, 2024 · 0 comments · Fixed by #201
Closed

my-page로 임의 접근 시 로그인 alert가 3번 발생하는 issue 해결 #186

jinyoung234 opened this issue Jun 13, 2024 · 0 comments · Fixed by #201

Comments

@jinyoung234
Copy link
Contributor

jinyoung234 commented Jun 13, 2024

as-is

2024-06-21.8.42.49.mov
  • /my-page로 임의 접근 시 로그인 alert가 총 3번 열리는 issue 발생
    • MyPage 컴포넌트 내 queryFn인 fetchBookmarkList, fetchUserProfile, fetchUserReviewList 내 accessToken이 없을시 alert로 처리 하는 로직이 원인
    • 즉, 각 fetcher의 alert 로직이 모두 실행 되는 것이 원인

to-be

  • fetchBookmarkList, fetchUserProfile, fetchUserReviewList의 window.alert 및 redirect 로직을 걷어낸 후 error를 throw 하도록 변경한다.
  • MyPage에서 access token 관련 에러가 발생했을 때 alert 3회 발생 후 redirect 되는 이슈를 해결한다.
  • BookmarkListPage에서 access token 관련 에러가 발생했을 때 alert 후 redirect 되도록 변경한다.
  • BookmarkMapPage에서 access token 관련 에러가 발생했을 때 alert 후 redirect 되도록 변경한다.
  • MyReviewListPage에서 access token 관련 에러가 발생했을 때 alert 후 redirect 되도록 변경한다.
@jinyoung234 jinyoung234 closed this as not planned Won't fix, can't repro, duplicate, stale Jun 13, 2024
@jinyoung234 jinyoung234 reopened this Jun 13, 2024
@jinyoung234 jinyoung234 changed the title test github & notion 연동 테스트 (이슈 할당 시 해당 이슈로 대체할 예정 입니다!) Jun 13, 2024
@jinyoung234 jinyoung234 changed the title github & notion 연동 테스트 (이슈 할당 시 해당 이슈로 대체할 예정 입니다!) alert & confirm 처리된 페이지들 모달 UI로 개선 Jun 21, 2024
@jinyoung234 jinyoung234 changed the title alert & confirm 처리된 페이지들 모달 UI로 개선 my-page로 임의 접근 시 로그인 alert가 3번 발생하는 issue 해결 Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment