Skip to content

[Feature/#37] 소셜 로그인 연동(Naver / Kakao)#39

Merged
Seojegyeong merged 2 commits intodevelopfrom
feature/#37
Feb 6, 2026
Merged

[Feature/#37] 소셜 로그인 연동(Naver / Kakao)#39
Seojegyeong merged 2 commits intodevelopfrom
feature/#37

Conversation

@Seojegyeong
Copy link
Collaborator

@Seojegyeong Seojegyeong commented Feb 6, 2026

#37

🚨 관련 이슈

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

1. 소셜 로그인 로직 커스텀 훅 분리 및 UI 연결

  • useSocialLogin 훅을 생성하여 소셜 로그인 요청 URL 생성 및 리다이렉트 로직 분리
  • 로그인 및 회원가입 페이지 핸들러 연결

2. 이메일 인증 리팩토링

  • 서버 expireIn 값을 반영해 타이머 동작하도록 개선

😅 미완성 작업

  • 추후 내 정보 조회 API 연동 시 Zustand 로그인 상태 업데이트 예정
  • 백엔드 네이버 및 카카오 소셜 로그인 리다이렉트 URL 연결 요청
  • 백엔드 내 정보 조회 API 구현 요청

📢 논의 사항 및 참고 사항

  • 네이버와 카카오 로그인의 경우 클라이언트 로직 구현은 완료되었으나, 현재 백엔드 서버 설정(리다이렉트 URI 등) 및 이슈로 인해 정상 동작하지 않습니다. 서버 조치 후 최종 확인이 필요
  • 구글 로그인은 추후 도메인 확보 후 연동 예정이므로 클릭 시 "준비 중입니다" 토스트 알림으로 처리해 두었습니다.

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

@Seojegyeong Seojegyeong requested review from YermIm and jjjsun February 6, 2026 03:50
@Seojegyeong Seojegyeong self-assigned this Feb 6, 2026
Copy link
Collaborator

@jjjsun jjjsun left a comment

Choose a reason for hiding this comment

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

P4: 확인했습니다!! 고생하셨어요!!

@YermIm
Copy link
Collaborator

YermIm commented Feb 6, 2026

P4: 확인했습니다 고생하셨습니다!!

@Seojegyeong Seojegyeong merged commit c3b1923 into develop Feb 6, 2026
1 check passed
@Seojegyeong Seojegyeong deleted the feature/#37 branch February 6, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants