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

[Refactor]: 소셜 회원가입 API 리팩토링 #158

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

dev-Crayon
Copy link
Contributor

🐬 요약

소셜 회원가입 API 리팩토링

👻 유형

PR의 유형에 맞게 체크해주세요!

  • 버그 수정
  • 기능 개발
  • 코드 스타일 수정 (formatting, local variables)
  • 리팩토링 (no functional changes, no api changes)
  • 빌드 관련 변경사항
  • CI 관련 변경사항
  • 문서 내용 변경
  • 기타... (다음 줄에 사유를 입력해주세요)

🍀 작업 내용

PR에 담긴 작업 내용을 작성해주세요!

  • Request body 검증성 추가
  • 비즈니스 로직 캡슐화
  • નાસ 처리

🌟 관련 이슈

PR과 관련된 이슈 번호를 작성해주세요!

close: #157

Request body 검증성 추가
비즈니스 로직 캡슐화
에러 처리

Related to: #157
@dev-Crayon dev-Crayon added 🌱 승헌 승헌 전용 라벨 🎃 Refactor 리팩토링 관련 라벨 ☠️ High Priority 높은 우선 순위 라벨 labels Jul 28, 2024
@dev-Crayon dev-Crayon requested a review from onpyeong July 28, 2024 05:18
@dev-Crayon dev-Crayon self-assigned this Jul 28, 2024
Copy link
Member

@onpyeong onpyeong left a comment

Choose a reason for hiding this comment

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

util 함수를 따로 빼서 코드가 더 이해하기 쉬워진 것 같습니다!

@dev-Crayon dev-Crayon merged commit 9066ec7 into develop Aug 15, 2024
1 check passed
@dev-Crayon dev-Crayon deleted the refactor/#157-authLoginRefactor branch August 15, 2024 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☠️ High Priority 높은 우선 순위 라벨 🌱 승헌 승헌 전용 라벨 🎃 Refactor 리팩토링 관련 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 회원가입 API 리팩토링
2 participants