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
[Feat] 소셜 로그인 토큰 UserDefault에 저장 및 기타 수정 (#203) #205
The head ref may contain hidden characters: "feat/#203-\uC18C\uC15C-\uB85C\uADF8\uC778-\uD1A0\uD070-UserDefault\uC5D0-\uC800\uC7A5"
[Feat] 소셜 로그인 토큰 UserDefault에 저장 및 기타 수정 (#203) #205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
역시 MVP...
@@ -7,9 +7,14 @@ | |||
|
|||
import UIKit | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// MARK: - Protocols
|
||
import Foundation | ||
|
||
@propertyWrapper struct UserDefaultWrapper<T> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
@@ -139,6 +138,16 @@ final class HomeController: BaseUploadViewController { | |||
// checkPostStatus() | |||
setActionSheet() | |||
} | |||
|
|||
func checkLogin() { | |||
if !userManager.hasJwtToken { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
클린코드를 지킵시다..~~
not 구문 사용 지양
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
클린코드 잘 공부했네요 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그만잘해
let createCharacterView = CreateCharacterViewController.instantiate() | ||
createCharacterView.isSignUp = false | ||
let nav = UINavigationController(rootViewController: createCharacterView) | ||
nav.modalPresentationStyle = .fullScreen | ||
present(nav, animated: true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instantiate 야무지게 잘 쓰셨네요 굳
그리고 nav는 뭐죠?? 약어 안쓰기로 했잖아요 ..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
걸렸네 😅
@@ -7,9 +7,14 @@ | |||
|
|||
import UIKit | |||
|
|||
protocol CustomPopUpControllerDelegate: AnyObject { | |||
func popUpDidDismiss() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
얘 이름 뭔가 이상한데? ㅋㅋ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넘어가줘
if newValue == nil { UserDefaults.standard.removeObject(forKey: key) } | ||
else { UserDefaults.standard.setValue(newValue, forKey: key) } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 린트 안걸렸나여?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와 귀신 같이 잡네..
인간 린트세요?
@@ -32,7 +36,10 @@ class CharacterNameViewController: UIViewController { | |||
|
|||
namingCharacterLabel.attributedText = attributedStr | |||
|
|||
self.dismiss(animated: true) | |||
guard let pvc = self.presentingViewController else { return } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p..v..c..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정하겠습니다!
self.dismiss(animated: true) | ||
guard let pvc = self.presentingViewController else { return } | ||
self.dismiss(animated: true) { | ||
pvc.dismiss(animated: true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To Do: 인디케이터바 추가하기
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
인디케이터바는 수빈님이 할래요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ㅇㅇ..
@@ -180,6 +180,7 @@ final class HaruHappicDetailController: UIViewController { | |||
|
|||
@objc private func showAlertPopUp() { | |||
let alartPopUpView = CustomPopUpController() | |||
alartPopUpView.delegate = self |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
setDelegate함수에 빼는거랑 showAlertPopUp함수에 넣는거랑 어떤게 맞을지 궁금함 (그냥 궁금)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alertPopUpView 가 showAlertPopUp 함수 내부에서 선언되어 있어서 setDelegate에서는 접근이 불가능해요 그래서 여기서 밖에 delegate 지정을 못해줍니당
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아하 넵!
💥 관련 이슈
💥 구현/변경 사항 및 이유
💥 PR Point
💥 참고 사항