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.

Sign up for GitHub

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

Jump to bottom

[FEAT/#7] kakao login #12

Merged
merged 8 commits into from
Dec 31, 2023
Merged

[FEAT/#7] kakao login #12

merged 8 commits into from
Dec 31, 2023

Conversation

chattymin
Copy link
Member

⛳️ Work Description

  • 로그인 UI 구현
  • activity, viewmodel 기본적인 셋팅
  • SDK 설정
  • 로그인 기능 구현
  • local.properties에 native.app.key 추가 및 적용
  • kakao 디벨롭 사이트에 키 해시 추가

📸 Screenshot

📢 To Reviewers

  • shared preferences는 Splash 에서 관리해야 한다고 생각하여 안만들었습니다.
  • 로그인 ui는 와이어프레임이기 때문에 추후 디자인이 나온다면 수정 될 예정입니다.
  • 개인정보 처리 약관 앞에 ! 아이콘이 없는 것도 위와 같은 이유입니다.
  • doamin 모듈에 있는 AuthTokenModel은 예시입니다. 서버와의 통신 방법을 결정 한 후 변경될 수 있습니다.
  • 대부분의 코드가 서버와의 통신이 결정되어야 작성할 수 있어 큰 틀만 작성해뒀습니다. 추후 수정할 예정입니다.
  • 모든 안드 팀원들의 키해시를 받아와서 카카오에 넣어야 합니다. 추후 각자 구해서 넣으시면 될 것 같습니다.
  • LoginActivity의 onCreate에 아래 내용을 넣으면 로그에 나옵니다.
    val keyHash = Utility.getKeyHash(this)
    Timber.tag(AUTH).d(keyHash)

@chattymin chattymin added 동민 🐥 FEAT ✨ 새로운 기능 구현 labels Dec 30, 2023
@chattymin chattymin assigned chattymin and unassigned chattymin Dec 30, 2023
Marchbreeze
Marchbreeze approved these changes Dec 31, 2023
View reviewed changes
Copy link
Member

@Marchbreeze Marchbreeze left a comment

Choose a reason for hiding this comment

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

아니 모 괴롭힐게 없네 ... ㅠㅠ
xml에서 constraintLayout 내부에 match_parent들만 수정해주셍요 ~

presentation/src/main/java/com/going/presentation/auth/LoginActivity.kt Outdated
observeInfo()
}

private fun initKakaoLoginButtonClickListener() {
Copy link
Member

Choose a reason for hiding this comment

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

요고 함수명 우리 ~ BtnClickListener 로 통일할까?

Copy link
Member Author

Choose a reason for hiding this comment

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

넵! 이런 부분 공유 아주 조아용~

presentation/src/main/java/com/going/presentation/auth/LoginActivity.kt
Comment on lines +31 to +34
private fun observeInfo() {
observeIsAppLoginAvailable()
observePostChangeTokenState()
}
Copy link
Member

Choose a reason for hiding this comment

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

두개정도면 따로 함수화 추가적으로 진행 안하고 onCreate에 observe 함수 두개 두는건 어떨까나요

Copy link
Member Author

Choose a reason for hiding this comment

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

이 부분은 앞으로 observe 할 값이 많아질 것이라고 예상돼서 묶어뒀습니다!
추후 observe할 데이터 갯수가 더 늘어나지 않거나 얼마 안된다면 onCreate에 합쳐두도록 하겠습니다 :)

presentation/src/main/res/layout/activity_login.xml Outdated

<TextView
android:id="@+id/tv_sign_in_title"
android:layout_width="match_parent"
Copy link
Member

Choose a reason for hiding this comment

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

match_parent 대신 0dp를 사용합시당 ~ 아마 그러면 android:gravity="center" 도 안해도 될듯

Copy link
Member Author

@chattymin chattymin Dec 31, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

이건 자꾸 까먹네요... 습관 들이게 노력하겠습니다 :)
근데 gravity center안하니까 왼쪽으로 가버리네요 ㅋㅋㅋ
센터 설정은 유지하겠습니당

presentation/src/main/res/layout/activity_login.xml
android:layout_marginHorizontal="16dp"
android:layout_marginBottom="8dp"
android:gravity="center"
android:lineHeight="42dp"
Copy link
Member

Choose a reason for hiding this comment

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

lineHeight 한줄로 설정해두는 것들도 typo에 넣어뒀으니 나중에 UI 수정할때 활용하면 좋을듯합니닷

Copy link
Member Author

Choose a reason for hiding this comment

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

이 부분은 추후에 디자인이 확정되고, 전체 수정할때 참고하도록 하겠습니다!
typo 만드느라 고생하셨습니당 🫶

@chattymin chattymin merged commit 0f281f9 into develop Dec 31, 2023
1 check passed
@chattymin chattymin self-assigned this Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Marchbreeze Marchbreeze Marchbreeze approved these changes

@leeeyubin leeeyubin Awaiting requested review from leeeyubin

@crownjoe crownjoe Awaiting requested review from crownjoe

Assignees

@chattymin chattymin

Labels
FEAT ✨ 새로운 기능 구현 동민 🐥
Projects
None yet
Milestone
1차 스프린트 서버통신 작업
Development

Successfully merging this pull request may close these issues.

[FEAT] OnBoarding 카카오톡 소셜로그인 구현
2 participants
@chattymin @Marchbreeze