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

[FE] feat : 로그인 및 회원가입 연동 #163

Open
wants to merge 3 commits into
base: merge/front
Choose a base branch
from

Conversation

eunjin11
Copy link
Collaborator

@eunjin11 eunjin11 commented Mar 6, 2025

#️⃣연관된 이슈

📝작업 내용

작업한 내용을 작성해주세요.

이메일 인증
->로그인
->로그인 결과 isMember이 false면 회원가입창으로 이동, 닉네임 입력받음
->회원가입 후 다시 로그인, 토큰 발급받아 localStorage와 cookie에 저장

로그인 시 response body로 userId, userName, email이 오는데 프론트에서 어떻게 활용하는지 자세히 모르겠어서는 따로 상태관리로 저장해두진 않았어요 userStore 등에서 필요하면 저장해서 쓰시면 될 것 같아요

스크린샷 (선택)

image

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

그리고 api/auth/sign-in에서 발급된 jwt에 userId가 없길래 로그인 후에 기존 토큰발급(로그인) api api/auth/login에서 토큰 새로 발급해서 넣었습니다..

그리고 member 서버 volume이 없어서 일부러 저장되고 곧 만료되게 해두신건가요??
그리고 가끔 이메일 인증코드 전송이 안되는 문제가 있는데 서버 껐다 키면 되더라구요

@eunjin11 eunjin11 requested review from yeona813 and pigpgw March 6, 2025 22:03
@eunjin11 eunjin11 self-assigned this Mar 6, 2025
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.

1 participant