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.

Sign up for GitHub

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

Jump to bottom

gh-2: 사용자 회원가입(API) 구현 #13

Merged
merged 6 commits into from
Nov 9, 2023
Merged

gh-2: 사용자 회원가입(API) 구현 #13

merged 6 commits into from
Nov 9, 2023

Conversation

JaeHyuckSa
Copy link
Contributor

@JaeHyuckSa JaeHyuckSa commented Nov 9, 2023 •
edited
Loading

PR 체크리스트

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

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

PR 유형

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

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

현재 동작은 무엇인가요?

이슈 번호: N/A
#2

새로운 동작은 무엇인가요?

이 PR은 호환성 변경을 도입하나요?

  • 예
  • 아니요

기타 정보

image

@JaeHyuckSa
Feat: User 모델 작성 #1 #2
2418250 
- user 모델 작성 및 manager 작성
Related to #1 #2
@JaeHyuckSa
Test: 회원가입API, 유저 서비스 및 선택자 테스트 코드 작성 #2
2ec3e1c 
- API 테스트
- 테스트 성공
- 테스트 실패 (비밀번호 유효성, 계정 존재, 빈값, 빈필드)
- 서비스 및 선택자 테스트
- 테스트 성공
Related to #2
@JaeHyuckSa
Feat: 회원가입 서비스 로직 및 API 작성 #2
b43ee56 
- 회원가입 API 및 서비스, 선택자 로직 작성
Related to #2
@JaeHyuckSa JaeHyuckSa self-assigned this Nov 9, 2023
@JaeHyuckSa JaeHyuckSa linked an issue Nov 9, 2023 that may be closed by this pull request
feature: 사용자 회원가입(API) #2
Closed
@JaeHyuckSa JaeHyuckSa merged commit 788ee86 into develop Nov 9, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/002-signup branch November 9, 2023 08:49
@simseulnyang
Copy link

코드 확인했습니다! 저에게는 익숙하지 않은 새로운 방식을 택하셨는데
이렇게도 작성할 수 있다는 사실을 알게 되어 하나 더 배우게 되었습니다. 감사합니다!
예산 및 지출 부분 작성하실 때 코드도 무척 궁금하고 기대되네요😊

@lfoyh6591
Copy link

lfoyh6591 commented Nov 11, 2023 •
edited by JaeHyuckSa
Loading

코드 확인했습니다. django에서 service 쓰는 것도 흥미롭네요!

@JaeHyuckSa
Copy link
Contributor Author

코드 확인했습니다! 저에게는 익숙하지 않은 새로운 방식을 택하셨는데 이렇게도 작성할 수 있다는 사실을 알게 되어 하나 더 배우게 되었습니다. 감사합니다! 예산 및 지출 부분 작성하실 때 코드도 무척 궁금하고 기대되네요😊

감사합니다 :)

@JaeHyuckSa
Copy link
Contributor Author

코드 확인했습니다. django에서 service 쓰는 것도 흥미롭네요!

코드가 단순하면 MTV도 괜찮지만 길어질수록 사람들과 협업할 때 Django에서 제공해주는 기능들이 추상적이라 코드 기여하는 것이 어렵더라구요 그래서 새롭게 적용해봤습니다 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Chestnut90 Chestnut90 Awaiting requested review from Chestnut90

@lfoyh6591 lfoyh6591 Awaiting requested review from lfoyh6591

@simseulnyang simseulnyang Awaiting requested review from simseulnyang

Assignees

@JaeHyuckSa JaeHyuckSa

Labels
feature test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feature: 사용자 회원가입(API)
3 participants
@JaeHyuckSa @simseulnyang @lfoyh6591