Skip to content

Conversation

@wibaek
Copy link
Member

@wibaek wibaek commented Sep 27, 2024

관련 이슈

작업 내용

내 정보 API에서 email 정보를 확인할 수 있게 email 필드를 추가했습니다.

특이 사항

리뷰 요구사항 (선택)

혹시 개발 진행하실때 auth 필요한 부분은 어떻게 진행하시나요? Kakao OAuth를 기반으로 하니 로컬 개발 환경에서는 토큰 얻어서 postman으로 테스트해보기는 어려워서 결국 test case만으로 검증하게 되네요...

개인적으로는 아래 글과 같이 테이블 구성해서 리팩토링 하고, 일반 email&PW 로그인과 다른 OAuth 로그인들도 구현해보고 싶네요!
https://rastalion.dev/%ED%9A%8C%EC%9B%90-%EA%B0%80%EC%9E%85-%EB%B0%8F-%EB%A1%9C%EA%B7%B8%EC%9D%B8%EC%9D%84-%EC%9C%84%ED%95%9C-%ED%85%8C%EC%9D%B4%EB%B8%94-%EC%84%A4%EA%B3%84/

Copy link
Member

@leesewon00 leesewon00 left a comment

Choose a reason for hiding this comment

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

코드 변경사항도 거의 없고, 문제될만한 부분도 보이지 않네요. 고생하셨습니다!

@leesewon00
Copy link
Member

저는 로컬에서 서버를 띄워서 테스트하다보니 로컬서버에서 토큰을 발급받고 툴로 테스트를 진행해보기는 하는데,
프론트측에서 어떻게 사용해야 용이할지 고민을 해봐야될 것 같네요.
로컬환경에서 위백님이 편하게 토큰을 이용한 코드검증을 해보려면 어떤 솔루션이 필요한지 제안해주셔도 좋을 것 같습니다.

회원가입,로그인 전반에서의 코드 리팩토링은 앱으로 디벨롭하는데 있어서도 꼭 필요한 측면이 될 것 같습니다.
회원쪽 도메인 지식이 부족하여 새로 모집한 개발자분과 논의하여 과제를 해결해나가도 좋을 것 같습니다.

@wibaek wibaek merged commit a5286fa into solid-connection:main Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

마이페이지 API에 이메일 정보 추가

2 participants