Skip to content

Conversation

@junghwaYang
Copy link
Contributor

이슈 번호

close #275

변경 사항 요약

  • 이슈 본문 확인 부탁드립니다
  • 기존 프로필 가져오는 로직과 인증 로직 구분하여 작성했습니다
  • 기존에는 프로필 생성이 안되면 비로그인으로 간주되었습니다. 하지만 방금 막 회원가입한 유저는 프로필이 없기때문에 해당 로직을 분리하여, 각자의 역할을 명확히 명시하였습니다
  • 로그인 관련 로직임으로 @solprime 예도님 확인하시면 바로 머지 진행하겠습니다

@junghwaYang junghwaYang added 🐛 버그 버그 관련 태그 💫 업데이트 머지 완료된 파일을 업데이트 작업 할때 📡 API API 관련 기능 🔄 커스텀 훅 커스텀 훅 기능 개발 labels Dec 28, 2024
@junghwaYang junghwaYang requested a review from solprime December 28, 2024 13:50
@junghwaYang junghwaYang self-assigned this Dec 28, 2024
@junghwaYang
Copy link
Contributor Author

junghwaYang commented Dec 28, 2024

  • 위키 수정하기 에러 발생
  • 새로고침 시 로그인 인증상태 풀림 에러

원인

  • 위키가 생성되어있지않은 유저는 비로그인으로 인증되어 위키수정이 불가능했음

해결

  • 위키가 생성되어있지않아도 로그인 토근이 로컬 스토리지에 있다면 위키 수정가능하게끔 변경

변경사항

  • handleQuizSuccess 함수의 경우 이미 로그인 인증이 완료된 상태로 모달이 열린 후 실행함수기때문에 비로그인/로그인 구분 조건이 필요하지않음

Copy link
Collaborator

@solprime solprime left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@junghwaYang junghwaYang merged commit c7f09a7 into develop Dec 28, 2024
1 check passed
@junghwaYang junghwaYang deleted the feature/#275_인증-상태와-프로필-요청-로직-분리 branch December 28, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 버그 버그 관련 태그 💫 업데이트 머지 완료된 파일을 업데이트 작업 할때 📡 API API 관련 기능 🔄 커스텀 훅 커스텀 훅 기능 개발

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

인증 상태와 프로필 요청 로직 분리

3 participants