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

[BSVR-71] 로그인/회원가입, JWT 구현 #34

Merged
merged 48 commits into from
Jul 18, 2024
Merged

[BSVR-71] 로그인/회원가입, JWT 구현 #34

merged 48 commits into from
Jul 18, 2024

Conversation

wjdwnsdnjs13
Copy link
Member

📌 개요 (필수)

  • 카카오를 사용한 로그인/회원가입을 진행할 수 있어요
  • JWT와 필터를 구현했어요

🔨 작업 사항 (필수)

  • 회원가입 API
  • 로그인 API
  • 닉네임 중복 검사 API
  • JWT
  • JWT Filter

⚡️ 관심 리뷰 (선택)

  • conflict가 발생했는데, 제 윈도우에선 되는 데, 다른 환경에서도 돌아가는 지 모르겠어요. 확인이 필요해요
  • 슬랙에 오늘 결산에 보낸 환경 변수들을 .env 파일안에 넣어주세요!
  • DDL 수정이 필요해요.
    • email, name, phone_number 을 nullable로 변경해야해요.
    • level을 nullable, default value 1 로 변경해야해요.
  • 애플리케이션 실행 후 API들이 정상 동작하는지 확인이 필요해요.

💻 실행 화면 (필수)

  • 회원 가입
    image

  • 로그인
    image

  • 닉네임 중복 확인
    image

# Conflicts:
#	domain/src/main/java/org/depromeet/spot/domain/member/Member.java
#	infrastructure/jpa/src/main/java/org/depromeet/spot/jpa/member/entity/MemberEntity.java
#	usecase/src/main/java/org/depromeet/spot/usecase/service/MemberService.java
Copy link

github-actions bot commented Jul 17, 2024

Test Results

44 tests  +5   44 ✅ +5   0s ⏱️ ±0s
16 suites +2    0 💤 ±0 
16 files   +2    0 ❌ ±0 

Results for commit 66d769d. ± Comparison against base commit 952a226.

♻️ This comment has been updated with latest results.

@pminsung12
Copy link
Collaborator

pminsung12 commented Jul 17, 2024

엄청나다 1400줄, 수고많았어😢 리뷰 시작한다잇!

@EunjiShin
Copy link
Collaborator

EunjiShin commented Jul 17, 2024

DDL 수정이 필요해요.
email, name, phone_number 을 nullable로 변경해야해요.
level을 nullable, default value 1 로 변경해야해요.

@wjdwnsdnjs13 요건 저번에 얘기한대로, 카카오톡 비즈앱 생성되면 너가 entity 설정 바꿀 계획이라는 뜻이 맞을까?!

@EunjiShin
Copy link
Collaborator

EunjiShin commented Jul 17, 2024

스크린샷 2024-07-17 오후 5 59 51

@wjdwnsdnjs13 ?! 여기 브랜치에서 스웨거 안돌아가는 이슈 발견..! 👀

@EunjiShin
Copy link
Collaborator

스크린샷 2024-07-17 오후 6 02 47

@wjdwnsdnjs13 알려준 가이드 라인 대로, idCode 발급 받은 다음 회원 가입 API 호출했는데 500 에러 발생..!

스크린샷 2024-07-17 오후 6 04 26 로그 보니까 NPE 이슈인데, 확인 부탁해!

@EunjiShin EunjiShin added the ❌ do not merge 아직 merge하면 안되는 상태 label Jul 17, 2024
@wjdwnsdnjs13
Copy link
Member Author

엄청나다 1400줄, 수고많았어😢 리뷰 시작한다잇!

ㅋㅋㅋ 하다 보니까 줄이 많아지더라...

@wjdwnsdnjs13
Copy link
Member Author

DDL 수정이 필요해요.
email, name, phone_number 을 nullable로 변경해야해요.
level을 nullable, default value 1 로 변경해야해요.

@wjdwnsdnjs13 요건 저번에 얘기한대로, 카카오톡 비즈앱 생성되면 너가 entity 설정 바꿀 계획이라는 뜻이 맞을까?!

응응 맞아!
지금 엔티티랑 kakao에서 userInfo 받아오는 Dto도 일단 지금은 null 값으로 받아오지만, 비즈앱 되면 금방 적용가능하게 해놨어!

@wjdwnsdnjs13
Copy link
Member Author

스크린샷 2024-07-17 오후 6 02 47 @wjdwnsdnjs13 알려준 가이드 라인 대로, idCode 발급 받은 다음 회원 가입 API 호출했는데 500 에러 발생..!

스크린샷 2024-07-17 오후 6 04 26 로그 보니까 NPE 이슈인데, 확인 부탁해!

헐... 오키오키 확인해볼게

@wjdwnsdnjs13 wjdwnsdnjs13 changed the base branch from dev to main July 17, 2024 10:01
@github-actions github-actions bot added size/L and removed size/XL labels Jul 17, 2024
@EunjiShin EunjiShin merged commit 50dcd96 into main Jul 18, 2024
6 checks passed
@EunjiShin EunjiShin deleted the feat/BSVR-71 branch July 18, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❌ do not merge 아직 merge하면 안되는 상태 ✨ Feature 기능 개발 size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants