Skip to content

Conversation

@yyypearl
Copy link
Contributor

@yyypearl yyypearl commented May 4, 2025

연관 이슈

close #165

개요

네이버 로그인 (DISCORD 연동 테스트)

✅ 작업 내용

  • 네이버 로그인
  • OauthButton 리팩토링
  • 최근 로그인 방식 안내
  • 마이페이지 이메일 아이콘 constant로 리팩토링
  • 회원가입 완료 페이지 min-height 삭제

🖥 구현 결과

  • 네이버 로그인 (최근 로그인 다시 적용했어요)
20250504_225747.mp4
  • 최근 로그인 방식 표시
20250504_223440.mp4

리뷰 요구사항


📝 기타 사항

  • Oauth와 관련된 것들 기존에는 함수나 state로 설정되었는데, constants로 별도로 관리해서 가져오게 수정했어요! SocialGoogle.tsxSocialKakao.tsx도 따로 둘 필요가 없을 거 같아, OauthButton.tsx 컴포넌트로 통일해서 switch문으로 타입에 따라 리다이렉트 구현했습니다.
  • 마이페이지 아이콘도 마찬가지로 constants 폴더에 OAUTH로 관리했습니다.
  • 최근 로그인 방식은 쿠키로 관리할 이유가 없어(중요 정보X, 서버에 필요한 정보X) 로컬스토리지를 추천하더라구요. 그래서 해당 방식으로 적용했어요.
  • OAuthType이랑 Provider로 소셜로그인 종류 타입 정의했습니다! 필요하면 import 해서 사용하면 될 거 같아요.
  • 카카오도 env 다른 소셜로그인과 비슷하게 변수명 변경했어요! 네이버랑 같이 노션에 업데이트 해두겠습니다.

@yyypearl yyypearl added ✨ feat 새로운 기능 추가 ♻️ refactor 코드 리팩토링 labels May 4, 2025
@yyypearl yyypearl requested a review from hyo-4 May 4, 2025 13:29
@yyypearl yyypearl self-assigned this May 4, 2025
@github-actions
Copy link

github-actions bot commented May 4, 2025

🎉 Deploy Preview

https://lettering-o8wplj94m-yyypearls-projects.vercel.app
여기에서 배포 결과 확인하기

@github-actions
Copy link

github-actions bot commented May 4, 2025

🎉 Deploy Preview

https://lettering-lk43aw3xi-yyypearls-projects.vercel.app
여기에서 배포 결과 확인하기

Copy link
Member

@hyo-4 hyo-4 left a comment

Choose a reason for hiding this comment

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

확인👍👍

@hyo-4 hyo-4 merged commit d40b25d into develop May 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 새로운 기능 추가 ♻️ refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 네이버 로그인

3 participants