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

M3-159 카카오로그인 프론트 UI, 서비스 #23

Merged
merged 18 commits into from
Jul 10, 2024

Conversation

koomin1227
Copy link
Member

작업 내용*

  • 카카오톡 로그인 화면 구현
  • 카카오톡 로그인 구현, 성공시 secure storage에 저장
  • 회원가입시 회원가입 스크린으로 이동

고민한 내용*

에러 처리

  • 카카오 로그인 아래에서 모든 발생하는 에러는 컨트롤러에서 잡아서 처리한다.
  • 잡은 에러는 로그인 페이지에서 alert 창을 띄우게 구현했다.

환경변수

iOS/Flutter/Secrets.xcconfig 파일을 만들어 info.plist 에 들어가는 카카오 native_secret 코드를 직접 넣지 않도록 했다.

로그인 플로우

  1. 카카오 로그인 버튼 클릭
  2. kakaoSdk 에서 카카오 로그인 후 토큰 반환
  3. access token으로 스프링 서버에 로그인 요청
  4. 성공시
  • 회원가입이 안되어있으면 회원가입 창
  • 회원가입이 되어있으면 main 창

리뷰 요구사항

  • 리뷰할 때 중점적으로 봐줬으면 하는 내용들

스크린샷

Simulator Screenshot - iPhone 15 Pro Max - 2024-07-09 at 17 54 50

@koomin1227 koomin1227 self-assigned this Jul 9, 2024
Copy link
Contributor

@qjvk2880 qjvk2880 left a comment

Choose a reason for hiding this comment

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

카카오 sdk 덕분인지 몰라도 한번에 흐름이 잘 보이네요!!

<category android:name="android.intent.category.BROWSABLE" />

<!-- "kakao${YOUR_NATIVE_APP_KEY}://oauth" 형식의 앱 실행 스킴 설정 -->
<!-- 카카오 로그인 Redirect URI -->
Copy link
Contributor

Choose a reason for hiding this comment

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

주석은 철웅 멘토님의 가르침을 따르시는 건가여?

'[Message]: ${dioException.message}'
'[Data] : ${dioException.response?.data}\n'
'----------------------------------------------\n',
);
Copy link
Contributor

Choose a reason for hiding this comment

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

에러를 보여주는 로그가 좋은것 같아요!

@koomin1227 koomin1227 merged commit e49426c into develop Jul 10, 2024
1 check passed
@koomin1227 koomin1227 deleted the feature/M3-159-kakaoLogin branch July 10, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants