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

fix: 회원가입 API 수정 (#388) #389

Merged
merged 10 commits into from
Apr 11, 2023
Merged

Conversation

hyun99999
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • base URL 수정
    • 기존의 서버주소는 주석 처리
  • 회원 가입 API 수정
  • GenericResponse 모델을 수정
    • 이로 인해 error 모델 추가 및 다른 Service 파일에서도 모델 수정 반영하였습니다.
  • AccessToken 모델 추가해서 액세스 토큰을 받도록 하였습니다.
  • 카카오와 애플 소셜 로그인 구분하는 socialType 추가
  • refreshToken 코드는 추후에 사용할 수 있으니 주석 처리
  • 키체인 주석 코드는 사용하지 논의하고 삭제
  • pod update

📮 관련 이슈

- result 로 넘어오는 데이터 프로젝트에서 수정하지 않고 사용
- code, message 모두 string
- 회원가입 API 에서 응답으로 액세스 토큰을 받기 위한 데이터 모델
- 카카오와 애플 소셜 로그인 구분하는 socialType 추가
- refreshToken 코드는 추후에 사용할 수 있으니 주석 처리
- 키체인 주석 코드는 사용하지 논의하고 삭제
@hyun99999 hyun99999 added Hyungyu 🐯 현규 교수님 작업 Fix 코드 수정 및 버그, 오류 해결 labels Mar 30, 2023
@hyun99999 hyun99999 requested a review from dlwns33 March 30, 2023 00:36
@hyun99999 hyun99999 self-assigned this Mar 30, 2023
@hyun99999 hyun99999 merged commit d2312b3 into TeamNADA:develop Apr 11, 2023
@hyun99999 hyun99999 deleted the feature/#388 branch April 11, 2023 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 코드 수정 및 버그, 오류 해결 Hyungyu 🐯 현규 교수님 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 회원가입 API 수정
2 participants