Skip to content

Conversation

@leohan6540
Copy link
Contributor

📄 PR 내용 요약

  • 디자인 수정
  • 토큰 중복로직 제거
  • fetch -> axios 인스턴스로 변경

✅ 작업 내용 상세

  • 카카오 로그인 디자인 수정했습니다.
  • 토큰을 중복 저장하는 로직을 제거했습니다.
  • api호출을 fetch에서 axios 인스터스를 사용하도록 변경했습니다.
  • 로그인 전까지 로딩 스피너가 보이도록 했습니다.

📸 스크린샷 (선택사항)

💬 참고 사항

카카오 로그인 한번씩 테스트 해주시면 감사하겠습니다!

📝 체크리스트

  • 기능이 정상 동작하는지 확인
  • 로컬 빌드/테스트 통과
  • 불필요한 코드/주석 제거

@leohan6540 leohan6540 self-assigned this Nov 27, 2025
@vercel
Copy link

vercel bot commented Nov 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
coworkers Ready Ready Preview Comment Nov 27, 2025 6:00am

@leohan6540 leohan6540 added 🎨 Style CSS, 퍼블리싱, 반응형 관련 수정 ( 기능 수정 x ) 🛠️ Refactor 코드 리팩터링, 구조 개선, 클린 코드 작업 labels Nov 27, 2025
Copy link
Contributor

@huuitae huuitae left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍 코멘트 확인해주세요

const router = useRouter();
const queryClient = useQueryClient();
const redirect_uri = process.env.NEXT_PUBLIC_REDIRECT_URI;
const kakao_redirect_uri = process.env.NEXT_PUBLIC_KAKAO_REDIRECT_URI;
Copy link
Contributor

Choose a reason for hiding this comment

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

기존 환경 변수가 변경된 것 같은데 변경한 이유가 있을까요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

기존에 구글 환경변수도 추가하면서 접두사로 google, kakao를 추가해 헷갈리지 않도록 했습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

아 알겠습니다 그럼 버셀도 변경해야겠네요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 ㅜ

@leohan6540 leohan6540 merged commit 96cfde2 into develop Nov 27, 2025
5 checks passed
@leohan6540 leohan6540 deleted the feat/google-signin branch November 27, 2025 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ Refactor 코드 리팩터링, 구조 개선, 클린 코드 작업 🎨 Style CSS, 퍼블리싱, 반응형 관련 수정 ( 기능 수정 x )

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants