Feature/mz 150 kakao login #19

yangsooplus · 2023-12-26T10:12:13Z

💡 Issue

Resolved: [Feat] 카카오 소셜 로그인 #14

🌱 Key changes

카카오 SDK를 이용한 카카오 가입/로그인
카카오 accessToken을 이용한 수수 가입/로그인
Encrypted-DataStore로 토큰 저장소 교체
적당히 뼈대만 구현한 UI (그러나 겁나 많은 파일)

✅ To Reviewers

코드를 실행하기 전에

Kakao 키 해시 등록
1-1. Kakao SDK Debug Key 발급방법을 참고하여 디버그 키를 발급합니다
1-2. 카카오 디벨롭스에 수수 노션 - 보안 페이지의 카카오 개발자 계정으로 로그인하여 SUSU - 플랫폼 - Android에 디버그 키를 등록합니다
local.property 내용 추가
내용은 슬랙에 메세지 드리겠습니다~

코드 읽기 전에

UI는 기능 동작을 실험하기 위한 최소치라고 봐주세요
Feature/mz 154 navigation setting #18 PR이 머지되고 난 develop에 rebase 했습니다.
설명이 필요할만한 부분에 코멘트를 달았습니다.

📸 스크린샷

KakaoTalk_20240102_153056748.mp4

수수 토큰 만료 여부 체크해야 함

mypage module에서도 로그아웃을 위해 접근 가능해야 함

feature 모듈에서 hiltViewModel() 사용을 위해 추가했습니다.

data module에 액세스 불가하여 Repository를 주입하지 못하는 문제

사용하는 플러그인들에서 자꾸 파일이 생성되어서 추가합니다. 이전 프로젝트들에서는 항상 idea 폴더를 통으로 뺐어요!

jinukeu

중복되는 내용은 제외했어요~!
한번 얘기해보면 좋을거같습니다!

jinukeu · 2024-01-02T07:50:41Z

data/src/main/java/com/susu/data/model/response/TokenResponse.kt

+    val refreshToken: String,
+)
+
+fun TokenResponse.toDomain() = Token(


toDomain 대신 toModel 어떠신가용??
Domain 모듈과는 좀 다른 느낌이라 ... ㅎㅎ

jinukeu · 2024-01-02T07:58:04Z

domain/src/main/java/com/susu/domain/usecase/LoginUseCase.kt

+        tokenRepository.saveTokens(
+            token = loginRepository.login(oauthAccessToken = oauthAccessToken),
+        )


token 저장하는 로직을 loginRepository에서 처리하는건 어떻게 생각하시나요?

이유는 다음과 같습니다.

token 저장 로직은 도메인보단 데이터 영역에 더 가깝다고 생각합니다. 도메인에는 앱의 핵심 비즈니스 규칙이 들어가야하는데, 토큰 저장은 데이터 영역에 가깝다고 생각하기 때문이에요.

token 저장 로직을 domain에 넣은 경우, 추후 token 방식이 아닌 session 방식으로 변경이 되었을때 data layer와 domain layer 모두 수정을 해줘야합니다. / 만약 token 저장 로직이 data layer에만 존재한다면 session 방식으로 변경되었을 때 data layer만 변경하면 되므로 유지보수에 더 유리하다 생각해요.

jinukeu · 2024-01-02T08:12:17Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/login/LoginViewModel.kt

+                    loginUseCase(oauthAccessToken = oauthAccessToken)
+                        .onSuccess {
+                            postSideEffect(LoginContract.LoginEffect.NavigateToReceived)
+                        }
+                        .onFailure {
+                            postSideEffect(LoginContract.LoginEffect.ShowToast(it.message ?: "수수 로그인 실패"))
+                        }


요놈은 별도의 함수로 한번 더 분리하는거 어떤가요? onSuccess가 2번 있으니까 가독성이 떨어지는 것 같아요. (개인 취향)

jinukeu · 2024-01-02T08:13:03Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpContract.kt

+import com.susu.core.ui.base.SideEffect
+import com.susu.core.ui.base.UiState
+
+sealed interface SignUpContract {


sealed interface로 한번 더 감싼 이유가 뭔가요?

그...그냥 MVI 참고한 포스트나 레포에서 묶어쓰길래 국룰인갑다 하고 썼어요

지난 리뷰에서 sealed interface면 충분하다 말씀하셨던게 Contract 라인이 아니라 Effect 라인이엇군요

jinukeu · 2024-01-02T08:13:17Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpContract.kt

+import com.susu.core.ui.base.UiState
+
+sealed interface SignUpContract {
+    sealed class SignUpEffect : SideEffect {


Suggested change

sealed class SignUpEffect : SideEffect {

sealed interface SignUpEffect : SideEffect {

jinukeu · 2024-01-02T08:13:52Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpScreen.kt

+    viewModel: SignUpViewModel = hiltViewModel(),
+    navigateToReceived: () -> Unit,
+) {
+    val uiState by viewModel.uiState.collectAsState()


Suggested change

val uiState by viewModel.uiState.collectAsState()

val uiState by viewModel.uiState.collectAsStateWithLifecycle()

jinukeu · 2024-01-02T08:14:04Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpScreen.kt

+    LaunchedEffect(key1 = viewModel.sideEffect) {
+        viewModel.sideEffect.collect { sideEffect ->


collectWithLifecycle

jinukeu · 2024-01-02T08:23:05Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/social/KakaoLoginHelper.kt

+    fun getAccessToken(
+        callback: (String?) -> Unit,
+    ) {
+        UserApiClient.instance.accessTokenInfo { _, error ->
+            if (error != null) {
+                callback(null)
+            } else {
+                // 토큰 유효성 체크 성공(필요 시 토큰 갱신됨)
+                callback(TokenManagerProvider.instance.manager.getToken()?.accessToken)
+            }
+        }
+    }


val accessToken = TokenManagerProvider.instance.manager.getToken()?.accessToken
그냥 이렇게하면 콜백 안써도 되지 않나요?

UserApiClient.instance.accessTokenInfo를 사용하는 이유가 없어보여요.

UserApiClient.instance.accessTokenInfo를 사용하면 Kakao SDK에서 accessToken 만료시 토큰 갱신 후에 callback을 실행합니다
그런데 accessTokenInfo에서 제공하는 데이터 중에는 accessToken이 없어요! (최대 의문)
오로지 TokenManagerProvider를 통해서만 accessToken에 접근할 수 있습니다!

정리하자면, accessToken 만료 시 갱신하고, 갱신된 accessToken를 반환하기 위함입니다

아하 이런 사정이 있었군요 ... 어쩐지 ...
확인했슴다!

jinukeu · 2024-01-02T08:25:27Z

feature/mypage/src/main/java/com/susu/feature/mypage/MyPageViewModel.kt

+package com.susu.feature.mypage
+
+import androidx.lifecycle.viewModelScope
+import com.kakao.sdk.user.UserApiClient


조금 복잡하지만 callback 형식으로 만들면 kakao sdk 의존성 제거 가능하지 않을려나요?
카카오 sdk 의존성이 viewmodel에 있으면 나중에 테스트 코드 짜기가 어려울 것 같아요. 물론 이 프로젝트에서 테스트코드를 짜진 않을테지만 테스트 코드 작성이 유리하게 ViewModel을 만드는 연습을 해보면 어떨까 하는 생각입니다!

jinukeu · 2024-01-02T09:18:06Z

feature/navigator/src/main/java/com/susu/feature/navigator/initialization/MainViewModel.kt

+    private val checkCanRegisterUseCase: CheckCanRegisterUseCase,
+) : BaseViewModel<MainContract.MainState, MainContract.MainEffect>(MainContract.MainState()) {
+    init {
+        if (!KakaoLoginHelper.hasKakaoLoginHistory()) {


KakaoLoginHelper 의존성 끊는거 어때요??

jinukeu · 2024-01-02T14:35:40Z

feature/mypage/src/main/java/com/susu/feature/mypage/MyPageViewModel.kt

+) : BaseViewModel<MyPageContract.MyPageState, MyPageContract.MyPageEffect>(MyPageContract.MyPageState) {
+
+    fun logout() {
+        UserApiClient.instance.logout { error ->


앗 다시 보니 UserApiClient 관련 코드만 Screen 쪽에서 실행한다면 의존성 제거 가능할거같아요

jinukeu · 2024-01-02T14:44:59Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpViewModel.kt

+                            name = uiState.value.name,
+                            gender = uiState.value.gender,
+                            birth = uiState.value.birth.toInt(),


Suggested change

name = uiState.value.name,

gender = uiState.value.gender,

birth = uiState.value.birth.toInt(),

name = currentState.name,

gender = currentState.gender,

birth = currentState.birth.toInt(),

jinukeu · 2024-01-02T14:46:54Z

feature/loginsignup/src/main/java/com/susu/feature/loginsignup/signup/SignUpViewModel.kt

+        intent { copy(birth = birth) }
+    }
+
+    fun signUp() {


https://devscb.tistory.com/130
얼리 리턴 써보시는거 어때요? indent가 너무 많아서 가독성이 떨어지는 것 같아요.

(다른 코드에서도 얼리 리턴 쓰면 더 보기 좋을거같아요~! (제 개인 취향 같긴합니다만 ...)

syb8200

merge 가시죱!

yangsooplus added 30 commits December 26, 2023 14:42

chore: Timber Tree 심기

d7cf053

feat: kakao user sdk 설정

9e628a0

feat: SplashContract 정의 후 SplashViewModel 구현

c7fabab

수수 토큰 만료 여부 체크해야 함

feat: kakao login 구현

136f96f

feat: 수수 auth api 인증

2570023

refactor: domain에 KakaoLoginProvider 인터페이스 추가

7001066

mypage module에서도 로그아웃을 위해 접근 가능해야 함

fix: 수수 auth 관련 누락 파일 추가

10497e5

fix: 잘못 입력한 api 명세 수정

2f08e65

fix: api 변경에 따른 user 프로퍼티 변경

1af5735

feat: SplashViewModel 수수 토큰 로직 추가

5c04416

refactor: TokenRepository 저장 시 파라미터로 Token 사용

f7cdfbd

chore: AuthRepository.kt -> AuthRepositoryImpl.kt rename

a08f9d7

chore: KakaoLoginProvider.kt -> KakaoLoginProviderImpl.kt rename

323935e

feat: LoginContract, LoginViewModel 추가

732dba5

feat: Kakao di Module 추가

e6ae3c3

chore: feature plugin에 hilt.navigation.compose 의존성 추가

0a42e76

feature 모듈에서 hiltViewModel() 사용을 위해 추가했습니다.

fix: Repository di 불가 오류 해결

230df72

data module에 액세스 불가하여 Repository를 주입하지 못하는 문제

feat: 카카오 accessToken 접근 코드 추가

2b9d07e

refactor: Splash와 Login을 Login에 통합

2972865

fix: Authenticator를 token 재발행을 authRepository로 이동함에 따라 수정

765bc7f

feat: 기능 테스트 UI 추가

9e17f10

fix: hilt 순환 참조 제거

5f72d06

chore: 카카오 로그인 테스트용 로그 추가

6960045

fix: api 명세 형식 틀린 부분 수정

7a2867e

fix: ActivityContext가 필요한 로그인 동작을 분리

5bcd5dd

fix: token 저장을 완료할 때까지 대기

cebb823

refactor: encrpted-datastore로 교체

7f09a87

fix: kakao token 만료 체크 후 수수 로그인 시도하도록 수정

2cc691a

feat: ApiService에 수수 회원 탈퇴 추가

5d533b9

feat: login, signup nested navigation 지정

ca2e3f5

yangsooplus added 11 commits December 31, 2023 22:16

chore: gitignore에 idea 폴더 추가

9f16e97

사용하는 플러그인들에서 자꾸 파일이 생성되어서 추가합니다. 이전 프로젝트들에서는 항상 idea 폴더를 통으로 뺐어요!

refactor: TokenEntity -> TokenResponse 네이밍 변경과 유효기간 프로퍼티 제거

67a4348

refactor: UserEntity -> UserRequest

449c04c

refactor: AuthRepository를 Login과 SignUp으로 분리

4ad776b

refactor: 로그인/회원가입 동작 관련 Usecase 추가

722b085

feat: SplashScreen Api 적용

60675d5

feat: 앱 실행 데이터 작업 중에 SplashScreen을 계속 보이도록 설정

5f5b40e

feat: Splash를 보여주고 난 뒤, 유저 데이터에 따라 화면 이동

e21b8a0

refactor: TestScreen의 동작을 MyPage로 옮김

8928f59

feat: 카카오 로그인 후 회원가입 이탈 유저는 회원가입 화면으로 이동

846ade5

chore: detekt가 apiKey를 불러올 때 null check 무시

d4e1d00

yangsooplus requested review from jinukeu and syb8200 January 2, 2024 07:45

yangsooplus force-pushed the feature/MZ-150-kakao-login branch from af0a8f1 to d4e1d00 Compare January 2, 2024 07:57

jinukeu requested changes Jan 2, 2024

View reviewed changes

jinukeu reviewed Jan 2, 2024

View reviewed changes

fix: 카카오계정 로그인 후 진행 안되는 문제 수정

d0c87bb

yangsooplus mentioned this pull request Jan 3, 2024

[Refactor] 로그인/회원가입 리팩토링 #29

Closed

jinukeu approved these changes Jan 3, 2024

View reviewed changes

syb8200 approved these changes Jan 3, 2024

View reviewed changes

yangsooplus force-pushed the feature/MZ-150-kakao-login branch 4 times, most recently from 61baaa5 to 25ef406 Compare January 3, 2024 17:56

chore: detekt 수정사항 반영

d6f69ac

yangsooplus force-pushed the feature/MZ-150-kakao-login branch from 25ef406 to d6f69ac Compare January 3, 2024 18:03

yangsooplus merged commit 89dd447 into develop Jan 3, 2024
1 check passed

jinukeu deleted the feature/MZ-150-kakao-login branch January 15, 2024 06:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/mz 150 kakao login #19

Feature/mz 150 kakao login #19

yangsooplus commented Dec 26, 2023 •

edited

Loading

jinukeu left a comment

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

yangsooplus Jan 2, 2024

yangsooplus Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

yangsooplus Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024

jinukeu Jan 2, 2024 •

edited

Loading

syb8200 left a comment

	sealed class SignUpEffect : SideEffect {
	sealed interface SignUpEffect : SideEffect {

	val uiState by viewModel.uiState.collectAsState()
	val uiState by viewModel.uiState.collectAsStateWithLifecycle()

		LaunchedEffect(key1 = viewModel.sideEffect) {
		viewModel.sideEffect.collect { sideEffect ->

Feature/mz 150 kakao login #19

Feature/mz 150 kakao login #19

Conversation

yangsooplus commented Dec 26, 2023 • edited Loading

💡 Issue

🌱 Key changes

✅ To Reviewers

코드를 실행하기 전에

코드 읽기 전에

📸 스크린샷

jinukeu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jinukeu Jan 2, 2024 • edited Loading

Choose a reason for hiding this comment

syb8200 left a comment

Choose a reason for hiding this comment

yangsooplus commented Dec 26, 2023 •

edited

Loading

jinukeu Jan 2, 2024 •

edited

Loading