Nestsoft-Team / DunJi-BackEnd

DunJi-BackEnd
2 stars 0 forks source link

[PR] Sign up, Univ #21

Closed jeeheaG closed 1 year ago

jeeheaG commented 1 year ago

변경된 카카오 로그인 & 회원가입 로직 반영과 대학교, 대학교 인증 기능 구현했습니다.

아직 예외 처리, 예외를 Handler로 핸들링하는 일과 대학교 이메일 인증 시 인증하는 이메일주소 도메인이 DB에 있는 대학교 도메인과 일치하는 지 확인하는 과정이 안되어있습니다 앞으로 구현할 예정입니다! AuthService에서 토큰 처리하는 메서드들은 다시한번 정리할 계획입니다.

jeeheaG commented 1 year ago

고생하셨습니다! 그리고 merge conflict해결하셔야 할거 같아요 제가 먼제 develop브랜치에 머지해서 그런거 같기도 하네요 ..

머지는 로컬에서 해결해두었습니다! 그런데 User 관련 파일들 수정하셨던데 어떤 게 필요하셨던 건지 알 수 있을까요? 전체적으로 원래 있던 로직을 수정한 건 아니신 것 같아서 제가 이해되는 선에서 합치기는 했는데 혹시 새로 만드신 부분을 지웠을까봐요

리뷰 감사합니다! 오늘 내로 반영해서 올리고 배포하겠습니다.

gyuturn commented 1 year ago

아 원래 채팅방 @onetoMany옵션 줘서 관리하려다가 굳이 그렇게 안하고 dao거치는게 더 맞는거 같더라구요 그래서 추가했다가 삭제를 다시 했었습니다!

jeeheaG commented 1 year ago

아 알겠습니다!