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

gh-3: 사용자 로그인 (API) 기능 구현 #14

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

JaeHyuckSa
Copy link
Contributor

PR 체크리스트

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

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

PR 유형

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

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

현재 동작은 무엇인가요?

#3
이슈 번호: N/A

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

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

  • 아니요

기타 정보

image

@JaeHyuckSa JaeHyuckSa self-assigned this Nov 9, 2023
@JaeHyuckSa JaeHyuckSa linked an issue Nov 9, 2023 that may be closed by this pull request
@JaeHyuckSa JaeHyuckSa merged commit 0960d03 into develop Nov 9, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/003-signin branch November 9, 2023 11:46
@lfoyh6591
Copy link

토큰 발급 이외에 로그인 기능은 따로 하지 않으신 것 같은데 이유가 있으실까요?

@JaeHyuckSa
Copy link
Contributor Author

토큰 발급 이외에 로그인 기능은 따로 하지 않으신 것 같은데 이유가 있으실까요?

simple jwt에서 제공해주는 기능이 과제 최소한의 기능 요건에 맞추는 것 같아 모듈을 그대로 사용했습니다 :) 테스트 또한 simple jwt에서 작성한 테스틐 코드가 더 정밀하기에 따로 작성해지 않았습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

featre: 사용자 로그인(API)
2 participants