-
Notifications
You must be signed in to change notification settings - Fork 0
[Fix] Auth Provider / 프로필 페이지 redirect 로직 개선 #294
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
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🎭 Playwright Report✨ E2E Test가 성공적으로 완료되었습니다. Test 요약 내용을 확인해주세요.
📊 Test Summary
📜 Test Details✅ Passed Tests (3)
|
🎨 Storybook Report✅ 변경 사항이 없습니다 모든 Story가 이전 빌드와 동일합니다.
|
📊 Coverage Report
📉 #294을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #294 +/- ##
===========================================
- Coverage 40.86% 40.33% -0.53%
===========================================
Files 209 209 0
Lines 9367 9381 +14
Branches 423 406 -17
===========================================
- Hits 3828 3784 -44
+ Misses 5539 5597 +58 영향받은 파일
|
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
wooktori
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아까 얘기해주셨던 부분이군요! 고생하셨습니다!!
📝 변경 사항
💡 Axios Redirect 로직 개선
❓ 상황
/login으로 redirect 되어야 함userGetMe) 할 때/login으로 redirect 되지 않아야 함❌ 문제
인증 필요한 API 요청 시
/loginredirect가 발생하지 않음✅ 해결
🔍 결과
/login으로 redirect 됨userGetMe) 할 때/login으로 redirect 되지 않음💡 Auth Provider 로직 개선
❓상황
❌ 문제
✅ 해결
hasRefreshToken)hasRefreshToken을 AuthProvider로 전달, true 일 경우에만 Auth Provider의 refresh 로직 실행🔍 결과
💡 프로필 페이지 로직 개선
❓ 상황
/mypage로 redirect 필요❌ 문제
userGetMe+refresh) 발생✅ 해결
isAuthenticated가 true인 경우에만 내 정보 요청 실행하도록 수정🔍 결과
userGetMe) 요청하지 않음🔗 관련 이슈
Closes #
🧪 테스트 방법
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요