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

[feat] 로그인 및 회원가입 기능 #11

Merged
merged 26 commits into from
Jan 1, 2024
Merged

[feat] 로그인 및 회원가입 기능 #11

merged 26 commits into from
Jan 1, 2024

Conversation

ziiyouth
Copy link
Member

@ziiyouth ziiyouth commented Dec 30, 2023 •
edited
Loading

Related Issue 🪢

Summary 🌿

  • RoomType/RoomType Converter를 member->post로 옮겼습니다.
  • 비밀번호 암호화 처리
  • major는 별도의 major DB에서 검색해서 정보 조회한 뒤 프론트에서 string으로 보내주는 방식 맞을까요? (일단 string 입력으로 구현함)
  • 회원가입 API
  • 로그인 API
  • 이메일 중복확인 처리 - 인증번호 API에 추가
  • 비밀번호 재설정 API

Before i request PR review 🧤

  • jwt 잘 썼는지 봐주세요.. 엉엉

@ziiyouth
[chore] #10 시큐리티 passwordEncoder 추가
53ab6b2
@ziiyouth
[rename] #10 RoomTypeConverter를 Post 도메인으로 변경
605162d
@ziiyouth
[remove] #10 Member 엔티티에 RoomTypeConverter 삭제
cfbc476
@ziiyouth
[rename] #10 RoomType을 Member 도메인으로 이동
1f30e8a
@ziiyouth
[rename] #10 RoomType/Converter를 Post 도메인으로 이동
c6632ab
@ziiyouth
[chore] #10 회원가입 URL webSecurityCustomizer에 추가
d2228eb
@ziiyouth
[feat] #10 User 엔티티에 빌더로 createUser 추가
380e2a4
@ziiyouth
[feat] #10 UserRepository 생성
dd6d30d
@ziiyouth
[feat] #10 ProfileImageType 생성
7fe6cab
@ziiyouth
[feat] #10 회원가입 request dto 생성
b8f3983
@ziiyouth
[feat] #10 회원가입 response dto 생성
995afff
@ziiyouth
[feat] #10 회원가입 API controller
827b6fb
@ziiyouth
[feat] #10 회원가입 API service
03999b7
@ziiyouth
[rename] #10 위치 변경으로 인한 import 수정
89123c2
@ziiyouth
[feat] #10 회원가입 response dto에 memberId 추가
15f3fd3
@ziiyouth
[feat] #10 로그인 request dto 생성
c7e5ad6
@ziiyouth
[feat] #10 로그인 response dto 생성
6c80e1c
@ziiyouth
[feat] #10 로그인 기능 service 구현
3744544
@ziiyouth
[feat] #10 로그인 기능 controller 구현
3ede859
@ziiyouth
[feat] #10 로그인 관련 Errorcode 추가
62e7fc7
@ziiyouth
[chore] #10 로그인 URL webSecurityCustomizer 추가
a0b7cbb
@ziiyouth
[feat] #10 인증번호 전송 시 이메일 중복 확인
ffe5ceb
@ziiyouth
[fix] #10 토큰 발급 로직 수정
f55185d
@ziiyouth ziiyouth added the FEAT 새로운 기능을 추가 label Dec 30, 2023
@ziiyouth ziiyouth self-assigned this Dec 30, 2023
@OJOJIN
Copy link
Contributor

OJOJIN commented Dec 31, 2023

jwt 토큰 발급하는 부분 저렇게 쓰면 될 것 같아요. 수고하셨습니다!

RyuKwanKon
RyuKwanKon approved these changes Dec 31, 2023
View reviewed changes
Copy link
Contributor

@RyuKwanKon RyuKwanKon left a comment

Choose a reason for hiding this comment

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

리뷰가 늦었네요.. 죄송합니다. 프로필 enum으로 관리하는게 너무 좋은 것 같아요.

OJOJIN
OJOJIN reviewed Jan 1, 2024
View reviewed changes
Copy link
Contributor

@OJOJIN OJOJIN left a comment

Choose a reason for hiding this comment

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

추가적인 코멘트 남겼습니다.

src/main/java/org/gachon/checkmate/domain/member/service/MemberService.java
Copy link
Contributor

Choose a reason for hiding this comment

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

음! 지금 생각해보니 refreshToken 발급받은거를 redis에 저장하는 부분이 빠져있는 것 같아요.

@ziiyouth ziiyouth merged commit e284a18 into develop Jan 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OJOJIN OJOJIN OJOJIN left review comments

@RyuKwanKon RyuKwanKon RyuKwanKon approved these changes

Assignees

@ziiyouth ziiyouth

Labels
FEAT 새로운 기능을 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] 로그인/회원가입 기능
3 participants
@ziiyouth @OJOJIN @RyuKwanKon