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

Feat #9 - 회원가입 API에 사용자 등록 로직 추가 #10

Merged
merged 5 commits into from
Jun 21, 2024

Conversation

DongwookKim0823
Copy link
Member

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 기능 변경/삭제
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

관련 이슈

이슈 번호: #9

어떤 동작은 무엇인가요?

  • 유저 전화번호를 기반으로 고객 테이블을 조회 후 비어있는 유저 필드를 채웁니다.

기타 정보

- customers, pet_kindergardens, reservations, tickets 앱 추가
- 콜백 로직에 유저 등록 로직 추가
Close #9
@DongwookKim0823 DongwookKim0823 added the ✨ Feature 기능 개발(API, 모듈 등) label Jun 18, 2024
@DongwookKim0823 DongwookKim0823 self-assigned this Jun 18, 2024
- get_queryset_by_phone_number_and_user_id_is_null, register_customer 메서드 docstring 수정
@JaeHyuckSa
Copy link
Member

JaeHyuckSa commented Jun 21, 2024

@DongwookKim0823 전체적으로 셀렉터와 서비스 부분에서 사용하지 않는 코드들이 많은 것 같습니다. 처음부터 코드가 많을 경우관리가 힘들기에 하나하나씩 붙여나가는 것은 어떨까요?

Copy link
Member

@JaeHyuckSa JaeHyuckSa left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)

@DongwookKim0823 DongwookKim0823 merged commit 5f5da6d into develop Jun 21, 2024
1 check passed
@DongwookKim0823 DongwookKim0823 deleted the feature/issue-009/user-register branch June 21, 2024 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발(API, 모듈 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants