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

[FIX] #144 - 카카오톡 소셜 로그인과 스플래시 뷰 연결, 수정 #150

Merged
merged 12 commits into from
Dec 15, 2021

Conversation

mini-min
Copy link
Member

@mini-min mini-min commented Dec 14, 2021

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 카카오톡 어플이 설치되지 않았을 때 웹으로 카카오톡이 열리는 로직 수정
  • 스플래시 어플에 붙이기 + 스플래시 시간 조정
  • 스플래시 상황에서 자동 로그인 처리 로직 추가하기
  • 엑세스, 리프레시 토큰을 UserDefaults로 보내는걸 제대로 수정
  • 로그인 서버 통신 코드 수정
  • 기타 SwiftLint 워닝 수정

📸 스크린샷

기능 스크린샷 기능 스크린샷
스플래시
(라이트)
Simulator Screen Recording - iPhone 12 mini - 2021-12-15 at 01 00 09 스플래시
(다크)
Simulator Screen Recording - iPhone 12 mini - 2021-12-15 at 01 04 54

📮 관련 이슈

@mini-min mini-min added Minjae 🐻‍❄️ 민재 강남1타강사 작업 P1 / Priority High 우선순위 높음 Fix 코드 수정 및 버그, 오류 해결 labels Dec 14, 2021
@mini-min mini-min self-assigned this Dec 14, 2021
Copy link
Member

@dlwns33 dlwns33 left a comment

Choose a reason for hiding this comment

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

수고했어용 민재~

Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

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

놓쳤던건데 로그인뷰컨에서 goToMain() 메서드 네이밍을 컨밴션에 맞춰서 presentToMain 으로 하면좋을거 같아여!

  • 스플래시 화면 구현 멋집니당!

Comment on lines 74 to 76
print("me() success.")
let email = user?.kakaoAccount?.email
self.postUserSignUpWithAPI(request: email!)
Copy link
Member

Choose a reason for hiding this comment

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

이외에도 해결됐다면 프린트문 정리해주면 좋을거 같아여!
이메일 아마 필수라서 괜찮겠지만 혹시 모르니 강제언래핑은 피해주세욥

Copy link
Member Author

Choose a reason for hiding this comment

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

프린트문 싹다 확인하고 정리하겠습니다 !

@@ -116,7 +116,7 @@ extension MoreViewController: UITableViewDelegate {
print(error)
} else {
self.makeOKAlert(title: "", message: "받은 명함이 초기화 되었습니다.")
let acToken = UserDefaults.standard.string(forKey: Const.UserDefaults.token)!
let acToken = UserDefaults.standard.string(forKey: Const.UserDefaults.accessToken)!
Copy link
Member

Choose a reason for hiding this comment

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

여기도 강제추출보단 옵셔널 바인딩을 통해서 사용해주세요!

@mini-min mini-min merged commit 0698331 into TeamNADA:develop Dec 15, 2021
@mini-min mini-min deleted the feature/#144 branch December 15, 2021 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 코드 수정 및 버그, 오류 해결 Minjae 🐻‍❄️ 민재 강남1타강사 작업 P1 / Priority High 우선순위 높음
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 카카오톡 소셜 로그인과 스플래시 뷰 연결, 수정
3 participants