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/로그인 시 계정 정보 저장 및 헤더 반영 #530 #533

Conversation

publdaze
Copy link
Contributor

@publdaze publdaze commented Aug 20, 2023

연관 이슈

작업 요약

  • 로그인 시 필요한 정보 localstorage에 저장하고 로그인 여부에 따른 헤더 처리해주고 로그아웃 기능 추가하였습니다.

작업 상세 설명

  • 로그인 성공 시 member atom 값 설정해주는 로직 UI와는 상관없는 부분이라 로그인 컴포넌트 쪽에서 API 쪽으로 옮겼습니다.
  • 로그아웃 시 member atom 값 비우고 메인페이지로 이동하도록 처리해주었습니다.

리뷰 요구사항

  • 예상 소요 시간 7분
  • 프로필 이미지 해당 user에 맞게 띄워주는 거 추가로 처리해야 합니다. 이슈 분리하여 진행 예정입니다 헤더 프로필 이미지 적용 #531
  • 프로필 이미지 클릭 시 뜨는 계정 정보 이메일이 아닌 아이디로 처리하고 싶은데 로그인 응답으로 아이디는 제공해주지 않아서 백엔드 측과 의논해볼 예정입니다. 아이디 받아와 띄워주는 걸로 변경 완료!

Preview 이미지

image image

@publdaze publdaze added the P2 우선순위 두번째 label Aug 20, 2023
@publdaze publdaze self-assigned this Aug 20, 2023
@publdaze publdaze added P1 급하게 머지해달라 요구할 때 사용 and removed P2 우선순위 두번째 labels Aug 21, 2023
Copy link
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@publdaze publdaze merged commit 470304c into develop Aug 23, 2023
@publdaze publdaze deleted the feature/로그인_시_계정_정보_저장_및_헤더_반영_#530 branch August 23, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 급하게 머지해달라 요구할 때 사용
Projects
Status: Achieve
Development

Successfully merging this pull request may close these issues.

로그인 시 계정 정보 저장 및 헤더 반영
2 participants