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

Login API 구현 #26

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Login API 구현 #26

merged 2 commits into from
Oct 31, 2023

Conversation

lfoyh6591
Copy link
Collaborator

PR 체크리스트

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

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

PR 유형

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

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

현재 동작은 무엇인가요?

이슈 번호: #8

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

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

  • 예
  • 아니요

기타 정보

image
image

@lfoyh6591
♻️Refact : PR comment에 따른 수정
552caf0
@lfoyh6591
✨Feat : Login API 구현 및 confirm 관련 오류 수정
9738e8d 
- login serializer, view, url 구현
- token refresh 생성
- JWT setting
- user confirm관련 테스트 코드 오류 수정
@lfoyh6591 lfoyh6591 self-assigned this Oct 30, 2023
@lfoyh6591 lfoyh6591 linked an issue Oct 30, 2023 that may be closed by this pull request
feature: 사용자 로그인(API) #8
Closed
Chestnut90
Chestnut90 approved these changes Oct 30, 2023
View reviewed changes
Copy link
Collaborator

@Chestnut90 Chestnut90 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

고생하셨습니다!😁

simseulnyang
simseulnyang approved these changes Oct 30, 2023
View reviewed changes
Copy link
Collaborator

@simseulnyang simseulnyang left a comment

Choose a reason for hiding this comment

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

코드 확인했습니다. 수고하셨습니다!

@JaeHyuckSa
Copy link
Contributor

고생하셨습니다 :)

@lfoyh6591 lfoyh6591 merged commit d16aa7d into develop Oct 31, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/issue-008 branch November 19, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simseulnyang simseulnyang simseulnyang approved these changes

@JaeHyuckSa JaeHyuckSa JaeHyuckSa approved these changes

@Chestnut90 Chestnut90 Chestnut90 approved these changes

Assignees

@lfoyh6591 lfoyh6591

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feature: 사용자 로그인(API)
4 participants
@lfoyh6591 @JaeHyuckSa @Chestnut90 @simseulnyang