Feature/#26 feat sns login by imseongwoo · Pull Request #30 · BCSDLab/JJBAKSA-ANDROID

imseongwoo · 2022-12-30T06:05:37Z

PR 개요

이슈 번호: 26

PR Checklist

Code convention을 잘 지켰나요?
테스트 코드를 추가했나요?

작업사항

작업사항의 상세한 설명

카카오,네이버,구글의 토큰을 받아온 후 해당 정보로 로그인 및 자동로그인 기능 구현

추가내용

구글로그인 파이어베이스 인증과정에서 할당량 관련 에러가 발생해 구글로그인은 추후 구현 예정입니다.

Added Reviewers, Assignees, Labels, Milestone

wnehdals · 2023-01-05T10:08:11Z

app/src/main/java/com/jjbaksa/jjbaksa/di/NetworkModule.kt

 @Module
 @InstallIn(SingletonComponent::class)
 object NetworkModule {
+    private val gson = GsonBuilder().setLenient().create()


해당 객체가 필요한 이유가 혹시 있나요?

Use JsonReader.setLenient(true) to accept malformed JSON 이라는 에러가 발생해서 gson 객체 설정 후 retrofit 객체에 추가하였습니다.

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginActivity.kt

wnehdals · 2023-01-05T10:14:09Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginActivity.kt

-                        Log.e(TAG, "카카오계정으로 로그인 실패", error)
-                    } else if (token != null) {
-                        Log.e(TAG, "카카오계정으로 로그인 성공 ${token.accessToken}")
+                CoroutineScope(Dispatchers.IO).launch {


비즈니스 로직은 다 viewmodel로 빼주세요

713500f 6e85a9f 에서 수정했습니다.

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginActivity.kt

wnehdals · 2023-01-05T10:23:37Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+    private val _loginState = SingleLiveEvent<LoginResult>()
+    val loginState: SingleLiveEvent<LoginResult> get() = _loginState
+
+    private val _isNaverSignUpSuccess = SingleLiveEvent<Boolean>()


로그인 화면에서 필요한것은

회원인가?
2-1 회원이 아니라면 회원가입을 시키고 로그인 시킨다
3-1 오류가 난다면 오류를 사용자에게 보여준다
2-2 회원이면 로그인 시킨다
3-2 오류가 난다면 오류를 사용자에게 보여준다
결국 끝에는 로그인 시킨다 or 오류를 사용자에게 보여준다 입니다.
즉 로그인 화면(Activity)는 로그인 성공, 실패만 알면 됩니다.
이것의 내용을 담은 라이브데이터만 액티비티가 observe하면 됩니다
그렇게 하면 변수를 줄일 수 있어 좋아보입니다.

wnehdals · 2023-01-05T10:26:03Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+    val googleSignUpId = "google"
+    val naverSignUpId = "naver"
+
+    companion object {


companion object 는 제일 밑에 작성해주세요

…into feature/#26-feat-sns-login

…nuse livedata

imseongwoo · 2023-01-06T08:02:12Z

피드백 반영 완료했습니다!

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginActivity.kt

wnehdals · 2023-01-06T10:26:42Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+        }
+    }
+
+    fun socialSignUp(id: String, email: String, nickname: String) {


로그인 시도시 아이디가 없다면 code값으로 3을 줍니다 code값으로 다음 할 일을 정하도록 수정해주세요
3이면 회원가입을 시키면 될 것 같습니다.

163ce27 에서 수정했습니다.

wnehdals · 2023-01-06T10:27:02Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+        }
+    }
+
+    fun isKakaoSocialIdExist() {


코드값으로 구분하면 필요없어 보이는 로직과 함수입니다.

wnehdals · 2023-01-06T10:28:11Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+        }
+    }
+
+    fun getCustomKakaoSignUpEmail(): String {


소셜로그인시 새롭게 아이디, 패스워드, 이메일을 만들어 서버로 보내는것을 서버동아리원가 협의된 내용일까요?

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/UserApiClientExt.kt

wnehdals · 2023-01-09T08:45:31Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+                when (it.erroMessage) {
+                    "3" -> {
+                        when (socialNum) {
+                            1 -> {


1,2 이렇게 작성하면 협업하는 사람은 무슨 뜻인지 모릅니다.
enum을 활용 혹은 val NAVER = 1 이런식으로 변수로 의미를 부여해주세요

2a69c96 에서 수정했습니다.

wnehdals · 2023-01-09T08:46:54Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+            repository.postLogin(account, BuildConfig.social_login_password, isAutoLogin.value!!) {
+                _loginState.value = it
+                when (it.erroMessage) {
+                    "3" -> {


"3" 이렇게 작성하면 협업하는 사람은 무슨 뜻인지 모릅니다.
enum을 활용 혹은 val NEED_SING_UP = "3", api 응답값을 담는 Enum, 혹은 object 파일 하나 만들어서 변수에 의미를 부여해주세요

마찬가지로 2a69c96 에서 수정했습니다.

wnehdals · 2023-01-09T08:48:55Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginViewModel.kt

+
+    fun socialSignUp(id: String, email: String, nickname: String, socialNum: Int) {
+        val signUpReq = SignUpReq(id, email, nickname, BuildConfig.social_login_password)
+        viewModelScope.launch {


회원가입 실패에 대한 대처가 필요해 보입니다. 만약 회원가입 실패했다면 사용자는 실패한줄도 모르고 계속 소셜로그인 버튼을 누를 수 있는 상황이 생길 것 같습니다.

wnehdals · 2023-01-09T08:54:14Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/social/SocialLoginActivity.kt

+        viewModel.loginState.observe(this@SocialLoginActivity) {
+            if (it != null) {
+                if (it.isSuccess) {
+                    goToMainActivity()


로그인 성공한 이후 뒤로가기하면 다시 로그인 화면이 나와서는 안됩니다. 뒤로가기시 로그인 화면이 나오지 않도록 처리해주세요

960849f 에서 수정했습니다.

imseongwoo added 7 commits November 21, 2022 16:34

#26 [fix] Fix retrofit error

d6cbb56

#26 [add] Add matchNaverAccount func in RegexUtil.kt

e23cb0f

#26 [add] Add kakao login

f5ecedc

#26 [add] Add naver login

50121cf

#26 [add] Add social auto login

ab3d673

#26 [chore] Extract string

3f71536

#26 [chore] Delete unuse library

09a63a5

imseongwoo requested a review from wnehdals December 30, 2022 06:05

imseongwoo added 3 commits December 30, 2022 15:26

#26 [mod] Modify code convention

2992479

#26 [add] Add getCustomKakaoSignUpEmail function

58bb6b6

#26 [chore] Delete unuse string value

01f45e2

wnehdals reviewed Jan 5, 2023

View reviewed changes

imseongwoo added 4 commits January 6, 2023 14:21

Merge branch 'develop' of https://github.com/BCSDLab/JJBAKSA-ANDROID …

5fc3dcd

…into feature/#26-feat-sns-login

#26 [chore] Move business logic to viewmodel

713500f

#26 [chore] Move naver login business logic to viewmodel and remove u…

6e85a9f

…nuse livedata

#26 [chore] Delete unused signup function and unused import

5b3b164

wnehdals reviewed Jan 6, 2023

View reviewed changes

imseongwoo added 6 commits January 7, 2023 15:02

#26 [chore] Refactoring login business logic

85f532e

#26 [chore] Delete unuse import

d9cc432

#26 [chore] Delete unuse file

9cd1f7f

#26 [chore] Modify social login code

163ce27

#26 [chore] Modify indentation

e70d4ac

#26 [chore] Modify indentation

e3bbfb4

wnehdals reviewed Jan 9, 2023

View reviewed changes

imseongwoo added 4 commits January 9, 2023 19:11

#26 [chore] Modify socialLogin function variable to meaningful

2a69c96

#26 [add] Add email authenticate

7fafaca

#26 [mod] Modify custom kakao email to user's kakao email

36a82cd

#26 [chore] Delete wildcard import

fb3ec36

imseongwoo added 7 commits January 10, 2023 15:28

#26 [add] Add onBackPressed() function

960849f

#26 [add] Add Event class

b260358

#26 [add] Add scalar convert factory

e6b6548

#26 [add] Add kakao login

599781e

#26 [chore] mod code convention

c644f85

#26 [chore] remove comment

42d79cd

#26 [chore] Add new line in SocialLoginUrlMapper.kt

cd02886

Conversation

imseongwoo commented Dec 30, 2022 • edited by skdud0629 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR 개요

PR Checklist

작업사항

작업사항의 상세한 설명

추가내용

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

imseongwoo Jan 6, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

wnehdals Jan 5, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

imseongwoo commented Jan 6, 2023

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

imseongwoo commented Dec 30, 2022 •

edited by skdud0629

Loading

imseongwoo Jan 6, 2023 •

edited

Loading

wnehdals Jan 5, 2023 •

edited

Loading