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

회원가입, 로그인 수정 #196

Merged
merged 2 commits into from
Nov 15, 2023
Merged

회원가입, 로그인 수정 #196

merged 2 commits into from
Nov 15, 2023

Conversation

1g2g
Copy link
Member

@1g2g 1g2g commented Nov 14, 2023

⚙️ PR 타입

  • Feature
  • Hotfix

✨ 기능 설명 or 🚨 문제 상황

fix: registration 정보 params로 넘기기

feat: 임시 로그아웃 구현

👨‍💻 구현 내용 or 👍 해결 내용

🎯 PR 포인트

📝 참고 사항

redirect 페이지에서 카카오 로그인 정보를 넘겨줄 때,
인증되지 않은 회원(Registration)은 '정보입력 페이지'에 params로 넘겨주는 방식으로 변경했습니다.
인증된 회원(Authenticated)는 기존 방식 대로 localstorage에 저장됩니다.

요약 : Registration type은 localstorage에 저장되지 않습니다.

❓ 궁금한 점

@1g2g 1g2g added the 기능 코드의 기능이 추가되거나 바뀌었습니다. label Nov 14, 2023
@1g2g 1g2g added this to the 4차 스프린트 milestone Nov 14, 2023
@1g2g 1g2g self-assigned this Nov 14, 2023
@1g2g 1g2g linked an issue Nov 14, 2023 that may be closed by this pull request
@1eecan
Copy link
Contributor

1eecan commented Nov 14, 2023

🎉 @dlwl98 님 랜덤 리뷰어로 당첨되셨습니다! 리뷰를 부탁드립니다. 🙏

@1eecan 1eecan requested a review from dlwl98 November 14, 2023 04:50
@dlwl98 dlwl98 merged commit eec79ff into dev Nov 15, 2023
@dlwl98 dlwl98 deleted the fix/#188-register-login branch November 15, 2023 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 코드의 기능이 추가되거나 바뀌었습니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

회원가입에서 로그인까지 리팩토링, 수정
3 participants