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

Feature/5 naver oauth login refactor #7

Merged
merged 10 commits into from
Oct 8, 2024

Conversation

karnelll
Copy link
Contributor

@karnelll karnelll commented Oct 7, 2024

#️⃣연관 이슈

#5

📝작업 내용

네이버 OAuth 로그인 기능을 구현하여 사용자가 네이버 계정을 통해 로그인할 수 있도록 하는 기능입니다. 이 기능은 네이버 API와의 연동을 통해 네이버 로그인 인증 과정을 처리하고, 인증 성공 시 사용자 정보를 받아와 세션을 생성하거나 백엔드와 연동해 추가적인 사용자 데이터를 처리합니다.

  • 불필요한 파일 삭제
  • 네이버 로그인 버튼 추가 (NaverLoginButton.tsx 파일)
  • 네이버 OAuth 콜백 처리 API 구현 (app/api/auth/callback/naver/route.ts 파일)
  • 네이버 로그인 후 사용자 정보 처리 및 대시보드 페이지 연결
  • 환경 변수 설정 (NEXT_PUBLIC_NAVER_CLIENT_ID, NEXT_PUBLIC_NAVER_CLIENT_SECRET, NEXT_PUBLIC_REDIRECT_URI)
  • 백엔드와의 연동을 통해 액세스 토큰을 전달하고 사용자 정보를 저장/처리
  • 백엔드가 구현되기 전 테스트를 위해 임시로 프론트엔드에서 토큰을 콘솔에 출력하는 방식으로 처리

📸스크린샷

추가 사항

백엔드가 아직 구현되지 않아, 네이버 OAuth 인증 후 얻은 액세스 토큰을 프론트엔드 콘솔에 출력하여 테스트를 진행했습니다. 이 테스트는 백엔드와의 연동이 완료되면 삭제하거나 수정될 예정입니다.

  • 네이버에서 반환된 액세스 토큰을 콘솔에 출력 (console.log('Access Token:', accessToken))
  • 백엔드 연동 후 토큰 처리 및 사용자 정보를 백엔드로 전달하는 로직 구현 예정

💬리뷰 요구사항(선택)

@karnelll karnelll requested a review from choihooo October 7, 2024 12:29
@karnelll karnelll self-assigned this Oct 7, 2024
Copy link

vercel bot commented Oct 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
pingping-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 7, 2024 1:07pm

Copy link
Member

@choihooo choihooo left a comment

Choose a reason for hiding this comment

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

LGTM👍

@karnelll karnelll merged commit e732850 into main Oct 8, 2024
3 checks passed
@karnelll karnelll deleted the feature/5-naver-oauth-login-refactor branch October 9, 2024 07:37
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.

2 participants