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

[Feat] 카카오 로그인 방식 변경 (#12) #39

Merged
merged 4 commits into from
Jan 28, 2025

Conversation

chaen-ing
Copy link
Member

@chaen-ing chaen-ing commented Jan 27, 2025

#️⃣연관된 이슈

#12

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

카카오 로그인 v2

  • 네이티브 앱 환경에서 카카오 로그인을 위한 코드를 작성
  • api/v2/auth/login/kakao로 컨트롤러 따로 작성
  • 시큐리티에 경로 추가 완료

유저 프로필 입력 관련 수정

  • 연령대 > 생년월일로 대체
  • 업종, 직무 String으로 받아서 enum 변환 후 저장

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@chaen-ing chaen-ing added ♻️ Refactor 코드 리팩토링 ✨ Feat 새로운 기능 추가 labels Jan 27, 2025
@chaen-ing chaen-ing requested a review from chanmin-00 January 27, 2025 16:34
@chaen-ing chaen-ing self-assigned this Jan 27, 2025
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

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

너무 수고하셨습니다!

}

@JsonCreator
public static Job from(String value) {
Copy link
Member

Choose a reason for hiding this comment

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

직종을 닉네임처럼 랜덤 문자열로 받는다고 들었는데 그러면 enum 이외의 값이 들어올 수 도 있을거 같아요 정확하지는 않아서 한번더 물어봐야 할 거 같습니다

그렇게 되면 그냥 String으로 해야할거 같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

프론트분들께 목록 이외의 값 안들어오는 것으로 확인했습니당 예외처리는 해두어서 괜찮을 것 같아욥!!

@chaen-ing chaen-ing merged commit fe95544 into develop Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링 ✨ Feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants