Conversation
kwonsaebom
requested review from
Leeyoonji23,
jisooooooooooo and
shinjigu
as code owners
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 실패 😵 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
| const handleLogin = () => { | ||
| setIsLoggedIn(true); | ||
| openModal(<LoginModal onClose={closeModal} />); | ||
| }; |
There was a problem hiding this comment.
p1) 여기서 setIsLoggedIn(true)가 바로 실행돼서 로그인 성공 여부랑 상관없이 로그인 상태로 전환되고 있는 것 같아요.
지금 로그인 모달 열리고 바깥 영역 클릭해서 닫으면 로그인 버튼이 유지되는데 X 버튼으로 닫으면 로그인 버튼이 사라지고 메뉴들이 나오는데 요거 해결이 필요해 보입니당
shinjigu
left a comment
shinjigu
left a comment
There was a problem hiding this comment.
CI 오류는 Home.tsx의 미사용 useNavigate만 정리하면 해결될 것 같아요 !! 그리고 로그인 연결 작업 정말 수고 많으셨습니다 😽😽😽
|
|
||
| const getAccessToken = async (code: string) => { | ||
| const redirectUri = import.meta.env.VITE_GOOGLE_REDIRECT_URI; | ||
| const redirectUri = import.meta.env.VITE_GOOGLE_REDIRECT_URI_LOCAL; |
There was a problem hiding this comment.
p3) 여기서 env 직접 읽지 말고 authConstants.ts의 GOOGLE_REDIRECT_URI를 재사용해서 환경값을 한 곳에서 관리하면 좋을 것 같아요 !!
There was a problem hiding this comment.
그렇네요 ..! 수정했습니다 !!
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
빌드 결과빌드 실패 😵 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 실패 😵 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
3 participants
💡 Summary
uri를 배포 주소에서 로컬 주소로 변경하고, 구글 로그인 api 코드 제거했던 부분 다시 연결했습니다.
✅ Tasks
👀 To Reviewer
요거 먼저 리뷰 받고 바로 이슈 파서 로그인 상태 관리 작업 진행하겠습니다!