Feat/#6 #7

heerucan · 2021-10-18T18:23:00Z

📌 관련 이슈

close #6

📌 변경 사항 및 이유

2차 세미나 과제 완료입니다~!

📌 PR Point

CustomButton을 컴포넌트로 따로 빼서 만들어 준 걸 많이 봐서 한 번 따라서 구현하려고 해봤는데 생각보다 유용하게 쓴 것 같지 않아서 고부분 중점적으로 봐주시면 좋겠습니다!
CustomButton 클래스는 배경색이 있는 버튼에 해당하는 놈인데,, 그냥 글자만 있는 버튼한테 적용을 하지 못했다는...
그렇다고 걍 고놈 클래스에 해당하는 인스턴스를 맹그러주자하니... 코드 길이는 거기서 거기인지라...
저번에 private 관련 질문에 대한 답변을 토대로 접근제한자?를 잘 써주려고 했는데 어떤지...
지금 현재 이 코드에서 어떻게 해야 더 잘했다고 옆동네 안들오이드 손배들한테까지 소문이 날 수 있을까??? 비법 전수 부탁 바람..^^🤞 🌬
아! 나도 코드 더 잘짜고 싶다!!!!!!!!

📌 참고 사항

😊😊질문 및 지적 대환영😊😊

마다가스카/쇼미더아요 스터디 과제로 파베로그인 코드가 있는데 흐린눈하고 봐주세요 / 질문해주셔도 좋음

Taehyeon-Kim

PERFECT
저는 요정도 리뷰 남겨보도록 하겠습니다.
다른 멋진 분들이 또 코멘트를 남길테니까요..
역시나 잘하십니다 ✨

Taehyeon-Kim · 2021-10-19T03:33:55Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UITextField + Extension.swift

    func addLeftPadding() {
-        let paddingView = UIView(frame: CGRect(x: 0, y: 0, width: 15, height: self.frame.size.height))
+        let paddingView = UIView(frame: CGRect(x: 0, y: 0, width: 14, height: self.frame.size.height))
        self.leftView = paddingView
        self.leftViewMode = .always
    }

    func addRightPadding() {
-        let paddingView = UIView(frame: CGRect(x: 0, y: 0, width: 15, height: self.frame.size.height))
+        let paddingView = UIView(frame: CGRect(x: 0, y: 0, width: 14, height: self.frame.size.height))
        self.rightView = paddingView
        self.rightViewMode = .always
    }


지금은 width에 고정적인 값을 사용하고 있는데, 파라미터로 수치를 넘겨줘서 사용하면 조금 더 사용성있어지지 않을까 생각합니다 :-)

func addLeftPadding(_ padding: CGFloat) { ... }

와오 그것도 적용!!할게요😊

Taehyeon-Kim · 2021-10-19T11:10:56Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UILabel + Extensioin.swift

+import UIKit
+
+extension UILabel {
+    func addSpacing(kernValue: Double, paragraphValue: CGFloat) {


파라미터의 의미가 궁금합니다. 👀
안그래도 라벨안의 행 간격을 어떻게 조정할지 고민하고 있었는데, 요것을 사용하면 되겠군요!?

@Taehyeon-Kim 자간, 행간 관련 익스텐션인데 사실.. 저도 고것과 관련해서 킹이 받던 찰나에.. 캐치미 익스텐션을.. 적용해버렸습니다...;;🥲

kernValue나 ParagraphValue 같은건 아마 앱내에서 스타일이 2,3개거나 1개로 통일되어있을텐데 - 파라미터 자체에 기본값을 줘도 괜찮을거같아요!!

func addSpacing(kernValue : Double = -0.05, paragraphValue : CGFloat = (기본값) { …} sampleLabel.addSpacing()

왜냐면 우리가 Letter Spacing -0.5%, Line Spacing 1.1.. 몇이다 몇이다 일일이 확인하고 외우기는 귀찮으니까요... (휴먼에러로 틀릴수도 있고!)

나중에 실제 프로젝트 가서는 문맥마다 Headline / Subtitle/ Body/ 등등 크게 몇가지로만 나뉠텐데 거기 값들을 미리 저장해서 빠르게 라벨 세팅하는것도 좋을거같아요!

@i-colours-u 오 그러네요 ! 바로 적용할게요!!! 😊😊

Taehyeon-Kim · 2021-10-19T11:21:35Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/Component/CustomButton.swift

+
+import UIKit
+
+class CustomButton: UIButton {


CustomButton 클래스는 배경색이 있는 버튼에 해당하는 놈인데,, 그냥 글자만 있는 버튼한테 적용을 하지 못했다는...
그렇다고 걍 고놈 클래스에 해당하는 인스턴스를 맹그러주자하니... 코드 길이는 거기서 거기인지라...

우선 이것이 정답은 아니니 참고만 해주시기 바랍니다. 좀 더 재사용성을 높이고 싶다면, 예를 들어 배경색이 있고 없고를 조정하고 그 밖의 설정을 외부에서 조정하고 싶다면 이 커스텀 클래스(CustomButton)에 프로퍼티를 둘 수 있을 것 같습니다.

var backgroundColor: UIColor? = .clear { didSet { updateUI() } } private func updateUI() { self.backgroundColor = backgroundColor }

이런 느낌으로 didSet 프로퍼티와 같이 사용하면 backgroundColor의 값을 변경했을 때 색상이 업데이트 되겠죠?

추가적으로 이니셜라이저를 이용해서 외부에서 인스턴스를 생성할 때 아예 background 색상을 지정해줄 수도 있을 것 같습니다.

convenience init(backgroundColor: UIColor) { self.backgroundColor = backgroundColor }

다음의 것들을 이용해서 조금 더 자유도 높은 코드를 작성할 수 있을 것 같아요 :-)

선배님 앞으로 받들어 모시겠습니다. 그러니 자주 와서 코리를 달아주시길.

Taehyeon-Kim · 2021-10-19T11:32:06Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/CompleteVC.swift

+    private let completeButton = CustomButton().then {
        $0.setTitle("확인", for: .normal)
-        $0.setTitleColor(.white, for: .normal)
-        $0.titleLabel?.font = .boldSystemFont(ofSize: 18)
        $0.backgroundColor = .mainBlue
-        $0.layer.cornerRadius = 10
+        $0.isUserInteractionEnabled = true
        $0.addTarget(self, action: #selector(touchupCompleteButton(_:)), for: .touchUpInside)
    }


아 요런 느낌 때문에 그런거군요..
한 가지 방법이 더 생각났는데, 버튼을 배경색 있는 버튼과 배경색 있는 버튼으로 사용한다고 치면
아래와 같이 열거형을 만들어주고 분기 처리해서 설정을 다르게 set해줘도 괜찮아 보입니다.

// CustomButton.swift enum ButtonType { case Base case Google } ... private func setupUI(_ type: ButtonType) { if type == .Base { // write code } else if type == .Google { // write code } }

// 이렇게도 사용이 가능하지 않을까 싶네요..? private let completeButton = CustomButton().then { $0.setupUI(.Base) }

Taehyeon-Kim · 2021-10-19T11:33:25Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/MainVC/MainTabVC.swift

+    private func setNavigationController(title: String,
+                                 unselectedImage: UIImage,
+                                 selectedImage: UIImage,
+                                 rootViewController: UIViewController) -> UINavigationController {
+        let nav = UINavigationController(rootViewController: rootViewController)
+        nav.tabBarItem.title = title
+        nav.tabBarItem.image = unselectedImage
+        nav.tabBarItem.selectedImage = selectedImage
+        nav.navigationBar.isHidden = true
+        return nav
+    }
+}


요렇게 함수로 처리하는 것 좋네요 👍
반복코드 줄이기 퍼펙트⭐️

저도 한 번에 처리하는 법 알아가요! 짱짱👍🏻👍🏻

코드 깔끔하네요👏

i-colours-u · 2021-10-19T14:26:11Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UILabel + Extensioin.swift

+import UIKit
+
+extension UILabel {
+    func addSpacing(kernValue: Double, paragraphValue: CGFloat) {


kernValue나 ParagraphValue 같은건 아마 앱내에서 스타일이 2,3개거나 1개로 통일되어있을텐데 - 파라미터 자체에 기본값을 줘도 괜찮을거같아요!!

func addSpacing(kernValue : Double = -0.05, paragraphValue : CGFloat = (기본값) { …} sampleLabel.addSpacing()

왜냐면 우리가 Letter Spacing -0.5%, Line Spacing 1.1.. 몇이다 몇이다 일일이 확인하고 외우기는 귀찮으니까요... (휴먼에러로 틀릴수도 있고!)

나중에 실제 프로젝트 가서는 문맥마다 Headline / Subtitle/ Body/ 등등 크게 몇가지로만 나뉠텐데 거기 값들을 미리 저장해서 빠르게 라벨 세팅하는것도 좋을거같아요!

i-colours-u · 2021-10-19T14:41:36Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/SignUpVC.swift

@@ -167,7 +159,7 @@ class SignUpVC: UIViewController {

        FirebaseAuth.Auth.auth().createUser(withEmail: email, password: pw) { [self] (result, error) in


저도 요건 종종 빼먹는 부분인데 ㅎㅎ;;
strong reference cycle 방지하기 위해서 클로저에서는
[self] 말고 [weak self]로 바꿔 쓰는걸 습관화하는게 좋을거같아요!!
아니면 나중에 SwiftLint로 강제 처리해도 좋구요?!

@i-colours-u 질문이 있는데 그러면 weak self를 쓰고
내부에

guard let self = self else { return } 를 써줘야 하나요?
왜냐하면 weak self를 써주는 순간 옵셔널 처리를 해주라고 오류 메시지가 떴거든요.

weak 사용하게 된다면 guard let 으로 옵셔널 바인딩 처리해서 진행하는 편이 권장될거에요!!

i-colours-u · 2021-10-19T14:50:16Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/SignUpVC.swift

@@ -167,7 +159,7 @@ class SignUpVC: UIViewController {

        FirebaseAuth.Auth.auth().createUser(withEmail: email, password: pw) { [self] (result, error) in


여기 코드만 아니라 나중에 비동기처리,통신작업할때
(result,error) in 과 같은 형태로 많이 작성하게 될텐데여

result error : true,false,

result error : true, true

result error : false, true

result error : false , false

요렇게 내려오는 result/error에 대해 케이스가 총 4개가발생 할 수 있게 됩니다

사실 우리가 매번 고려하는건 성공이냐, 실패냐를 구분하기 때문에 1,3번 케이스정도만 고려하겠지만 -
2번 4번과 같이 애매한 상태가 되는 것(에러가 발생하면서 데이터가 넘어옴, 에러가 발생하지도 않고, 데이터도 없고 )도 원래는 고려를 해야하는 상황이 있을거에요

그래서 일일이 다 분기처리 하기보다는

Swift5 에서 기본적으로 제공하는
Result<Value,Error>문법을 활용하면 더 깔끔하게 작성이 가능할것 같아요!!

func auth(then handler: @escaping (Result<Data?,Error>) -> Void) { // 필요한 부분을 handler 에 담아서 escaping으로 넘겨주기 } auth { [weak self] result in result.success { data in // 성공했을 때 ~~ 처리 }.catch { error in // 에러 발생했을때 ~~ 처리, 필요에 따라 에러 분기처리하기 } }

extension Result { @discardableResult func success(_ successHandler: (Success) -> Void) -> Result<Success, Failure> { if case .success(let value) = self { successHandler(value) } return self } @discardableResult func `catch`(_ failureHandler: (Failure) -> Void) -> Result<Success, Failure> { if case .failure(let error) = self { failureHandler(error) } return self } }

(요거는 위에 Result 결과를 success, catch 형태로 간결하게 처리하기 위해 사용하는 Extension 입니다!!)

사실 지금같이 간단한 통신일때는 크게 상관없겠지만, 이후에 계속 해당 코드를 작성할때 다음과 같이 Result 문을 활용하면 성공/실패에 대해서만 고려할 수 있기 때문에 더 안전하게,직관적으로 작성이 가능할 것 같아요!!!

hyesuuou

슨배야 고생햇설 코드구우우우웃

hyesuuou · 2021-10-22T11:56:07Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Constant/Constant.swift

-        static let Description = "YouTube도 이동하며 계속하세요. 앱 및 Safari에서도 Google 서비스에 로그인됩니다."
+        static let Description = "YouTube도 이동하며 계속하세요.\n앱 및 Safari에서도 Google 서비스에 로그인됩니다."
    }

    struct Image {
        static let check = UIImage(systemName: "checkmark.square.fill")
        static let uncheck = UIImage(systemName: "square")
+        static let logo = UIImage(named: "logo")
+        static let homeFill = UIImage(named: "homeIconFill")
+        static let home = UIImage(named: "homeIcon")
+        static let shortsFill = UIImage(named: "shortsIconFill")
+        static let shorts = UIImage(named: "shortsIcon")
+        static let plueCircle = UIImage(named: "plueCircleIcon")
+        static let subscribeFill = UIImage(named: "subscriptionsIconFill")
+        static let subscribe = UIImage(named: "subscriptionsIcon")
+        static let libraryFill = UIImage(named: "LibraryIconFill")
+        static let library = UIImage(named: "LibraryIcon")


퍼가요~🧡

그.... 정말...궁금해서 질문합니다...!
아이콘들을 따로 파일을 만들어서 구조체에 넣어서 쓰면 언제 어떻게 좋은지 궁금합니다.....!

@hellozo0 저 복잡한 아이콘 파일명을 매번 쓰지 않아도 되고 복붙해서 가져다 쓰는 경우 간혹가다가 오타가 나는 경우도 있어요.
그럴 경우 앱이 크래시가 나는데 요론 문제들을 방지할 수 있어서 씁니다!

또 만약 checkmark 이미지 에셋의 이름이 바뀌었다고 하면 모든 파일에 들어가서 하나하나 바꿔야 하는 번거로움이 있는데
Constants로 만들어두면 한 번만 바꿈 되겠져? 고런 장점...쓰..?

사실 저도 이제 막 쓰기 시작했어요!!

hyesuuou · 2021-10-22T11:56:42Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UILabel + Extensioin.swift

+    func addColor(font: UIFont, range: String) {
+        if let labelText = text, labelText.count > 0 {
+            let attributedStr = NSMutableAttributedString(string: labelText)
+            attributedStr.addAttribute(.font,
+                                       value: font,
+                                       range: (labelText as NSString).range(of: range))
+            attributedText = attributedStr
+        }


쇽샥..해감니다 ><

요게 어떤 역할을 하는 함수인지 이해가 안가서,, 설명 한번만 해주세여,,,

@jane1choi 저도 코드 보니까 저 함수를 쓰진 않아서 왜 넣어줬는지 저도 의문이지만..;;
게다가 함수 이름도 잘못 써놨네요;; 진짜 왜 써줫지..??? 빼야겟당...

NSMutableAttributedString 은 텍스트의 특정 부분의 색이나 글자 크기, 폰트 등의 특성을 바꿔줄 때 사용하는 메소드에요.

//내가 적용하고싶은 폰트 사이즈 let fontSize = UIFont.boldSystemFont(ofSize: 30) //label에 있는 Text를 NSMutableAttributedString으로 만들어준다. let attributedStr = NSMutableAttributedString(string: myLabel.text) //위에서 만든 attributedStr에 addAttribute메소드를 통해 Attribute를 적용. kCTFontAttributeName은 value로 폰트크기와 폰트를 받을 수 있음. attributedStr.addAttribute(.foregroundColor, value: UIColor.blue, range: (text as NSString).range(of: "Zedd")) //최종적으로 내 label에 속성을 적용 myLabel.attributedText = attributedStr

아래 사진처럼 나오게 됩니당.!

출처: https://zeddios.tistory.com/300 [ZeddiOS]

hellozo0

후릐슨배 멋잇자나...?

hellozo0 · 2021-10-22T13:11:47Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Constant/Constant.swift

-        static let Description = "YouTube도 이동하며 계속하세요. 앱 및 Safari에서도 Google 서비스에 로그인됩니다."
+        static let Description = "YouTube도 이동하며 계속하세요.\n앱 및 Safari에서도 Google 서비스에 로그인됩니다."
    }

    struct Image {
        static let check = UIImage(systemName: "checkmark.square.fill")
        static let uncheck = UIImage(systemName: "square")
+        static let logo = UIImage(named: "logo")
+        static let homeFill = UIImage(named: "homeIconFill")
+        static let home = UIImage(named: "homeIcon")
+        static let shortsFill = UIImage(named: "shortsIconFill")
+        static let shorts = UIImage(named: "shortsIcon")
+        static let plueCircle = UIImage(named: "plueCircleIcon")
+        static let subscribeFill = UIImage(named: "subscriptionsIconFill")
+        static let subscribe = UIImage(named: "subscriptionsIcon")
+        static let libraryFill = UIImage(named: "LibraryIconFill")
+        static let library = UIImage(named: "LibraryIcon")


그.... 정말...궁금해서 질문합니다...!
아이콘들을 따로 파일을 만들어서 구조체에 넣어서 쓰면 언제 어떻게 좋은지 궁금합니다.....!

jane1choi

코드리뷰 넘 늦게 남겨서 죄송해여..
이번 주도 손배 코드보고 반복되는 코드 줄이는 방법과 좋은 네이밍에 대해 더더 배워갑니다☺️😆
고생많으셨어요👏👏

jane1choi · 2021-10-26T10:13:02Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UILabel + Extensioin.swift

+    func addColor(font: UIFont, range: String) {
+        if let labelText = text, labelText.count > 0 {
+            let attributedStr = NSMutableAttributedString(string: labelText)
+            attributedStr.addAttribute(.font,
+                                       value: font,
+                                       range: (labelText as NSString).range(of: range))
+            attributedText = attributedStr
+        }


요게 어떤 역할을 하는 함수인지 이해가 안가서,, 설명 한번만 해주세여,,,

jane1choi · 2021-10-26T10:15:43Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/CompleteVC.swift

    }

-    @objc func touchupLogoutButton(_ sender: UIButton) {
-        do { // 로그아웃
+    @objc func touchupAnotherAccountButton(_ sender: UIButton) {


네이밍 배워갑니다..👍🏻

jane1choi · 2021-10-26T10:18:57Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/MainVC/MainTabVC.swift

+    private func setNavigationController(title: String,
+                                 unselectedImage: UIImage,
+                                 selectedImage: UIImage,
+                                 rootViewController: UIViewController) -> UINavigationController {
+        let nav = UINavigationController(rootViewController: rootViewController)
+        nav.tabBarItem.title = title
+        nav.tabBarItem.image = unselectedImage
+        nav.tabBarItem.selectedImage = selectedImage
+        nav.navigationBar.isHidden = true
+        return nav
+    }
+}


저도 한 번에 처리하는 법 알아가요! 짱짱👍🏻👍🏻

jane1choi · 2021-10-26T10:26:05Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/SignUpVC.swift

@@ -46,7 +44,7 @@ class SignUpVC: UIViewController {
        $0.setTextField(placeholder: "비밀번호 입력", secure: true)
    }

-    lazy var showButton = UIButton().then {
+    private lazy var showButton = UIButton().then {


lazy 키워드는 언제 쓰는 것인지..🤔 설명해주시면 감사하겠습니다 슨배..

사실 저두 lazy var와 아주 사이가 친한 것이 아니라 여기저기에 갈겨보며 혼자 친한 척 좀 하는 중이긴 한데...
그래두 갈겨보자면...

lazy var는 모든 뷰가 다 생성(초기화)되고 나서 불리는 변수여요.
그러니까 당장 필요하지 않으면 생성되지 않고 호출을 해줘야만 생성이 되기 때문에 주로 가장
저기 showButton, fieldStackView 에는 굳이 쓸 필요 없는데 써 본 것이구

언제 사용하냐면
https://www.notion.so/huree-can-do-it/1-4697482733f246c98b2b7711d2e71859#7af5fe5b971d4bc5b46e054b35e42e5c
요기 참고해보쇼라~^^

jumining

Extension의 여왕 루희언니 코드 잘 보고가욥😀 나도 언젠간 익스텐션 잘 쓰게되는 날이 오길ㅜㅅㅜ

jumining · 2021-10-26T11:14:31Z

Sopt29th-Assignment/Sopt29th-Assignment/Resource/Extension/UIColor + Extension.swift

    @nonobjc class var mainBlue: UIColor {
      return UIColor(red: 66/255.0, green: 133/255.0, blue: 244/255.0, alpha: 1.0)
    }
+
+    @nonobjc class var lineGray: UIColor {
+      return UIColor(white: 234.0 / 255.0, alpha: 1.0)
+    }


non을 붙여야하나용,.? objc는 안되는건가요 궁금합니다!

저도 궁금해서 찾아봤는데
@nonobjc는 명시적으로 컴파일러가 해당 코드를 Objective-C로 변환하는걸 막고,
@objc는 명시적으로 컴파일러가 해당 코드를 Objective-C로 변환하도록 해준대요.

보통 전자를 사용하는 이유는 swift에서만 액세스할 수 있게 할 때 라는데
굳이 @nonobjc랑 @objc를 안 붙여도 될 거 같다는 생각이 드네요... 대신에 public을 붙여도 좋을 것 같아요.

그러면 왜 붙였냐?!!!
그건...그동안 제플린에서 쇽샥 색을 추출하면 저렇게 보통 코드값이 나와서 긁어왔던 버릇이 있어서..

jumining · 2021-10-26T11:16:28Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/CompleteVC.swift

-    @objc func touchupLogoutButton(_ sender: UIButton) {
-        do { // 로그아웃
+    @objc func touchupAnotherAccountButton(_ sender: UIButton) {
+        do {
            try FirebaseAuth.Auth.auth().signOut()
            guard let pvc = presentingViewController as? UINavigationController else { return }
            self.dismiss(animated: true) {


로그아웃 함수 보고 배워갑니당..!

jumining · 2021-10-26T11:17:56Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/LoginVC.swift

@@ -18,31 +18,28 @@ class LoginVC: UIViewController {
    // MARK: - Properties
    fileprivate var currentNonce: String?


이건 어떤건가용?

아 요거는 따로 스터디하면서 파이어베이스로 애플로그인을 구현해주려고 선언해준 건데

nonce 라는 게 임의로 생성되는 암호화 토큰으로 재생 공격을 방지하는 데 사용하는데
매번 애플로그인 요청 시에 nonce를 생성해서 보안을 해준다고 해요.

이 currentNonce를 그 역할을 하게 해주려고 만들었습니다. 사실 파베 도큐먼트에서 그렇게 하라고 하길래 긁어왔어여..ㅋㅋ

jumining · 2021-10-26T11:20:21Z

Sopt29th-Assignment/Sopt29th-Assignment/Source/View Controller/MainVC/MainTabVC.swift

+    private func setNavigationController(title: String,
+                                 unselectedImage: UIImage,
+                                 selectedImage: UIImage,
+                                 rootViewController: UIViewController) -> UINavigationController {
+        let nav = UINavigationController(rootViewController: rootViewController)
+        nav.tabBarItem.title = title
+        nav.tabBarItem.image = unselectedImage
+        nav.tabBarItem.selectedImage = selectedImage
+        nav.navigationBar.isHidden = true
+        return nav
+    }
+}


코드 깔끔하네요👏

heerucan added 9 commits October 18, 2021 23:53

🍱[ADD] 아이콘 에셋 추가(#6)

7ed8b3a

✅[CHORE] 이미지 에셋 Constant 추가(#6)

e0cbf13

➕[ADD] UILabel + Extension 추가(#6)

d744b5e

✅[CHORE] LoginVC layout 수정(#6)

1140753

✅[CHORE] SignUpVC layout 수정(#6)

f35aae6

✅[CHORE] CompleteVC layout 수정(#6)

d53a34d

✨[FEAT] UITabbar 구현 완료(#6)

c740dfe

➕[ADD] CustomButton Component 생성(#6)

599689d

✅[CHORE] Button 코드 수정 및 접근제한자 설정(#6)

f9a1bab

heerucan added 😊 huree 😊 assignment give me a code review 👻 labels Oct 18, 2021

heerucan requested review from YoonAh-dev, hryeong66, i-colours-u, Taehyeon-Kim, jane1choi, hyesuuou, dlwns33, jumining and yangsubinn October 18, 2021 18:23

heerucan self-assigned this Oct 18, 2021

Taehyeon-Kim approved these changes Oct 19, 2021

View reviewed changes

i-colours-u reviewed Oct 19, 2021

View reviewed changes

heerucan merged commit f9a1bab into main Oct 22, 2021

hyesuuou reviewed Oct 22, 2021

View reviewed changes

hellozo0 reviewed Oct 22, 2021

View reviewed changes

jane1choi reviewed Oct 26, 2021

View reviewed changes

jumining reviewed Oct 26, 2021

View reviewed changes

heerucan added a commit that referenced this pull request Oct 27, 2021

♻️[[REFACTOR] 2차 과제 리팩토링(#7)

08149a1

		@@ -167,7 +159,7 @@ class SignUpVC: UIViewController {

		FirebaseAuth.Auth.auth().createUser(withEmail: email, password: pw) { [self] (result, error) in

		@@ -18,31 +18,28 @@ class LoginVC: UIViewController {
		// MARK: - Properties
		fileprivate var currentNonce: String?

Feat/#6 #7

Feat/#6 #7

Conversation

heerucan commented Oct 18, 2021

📌 관련 이슈

📌 변경 사항 및 이유

📌 PR Point

📌 참고 사항

😊😊질문 및 지적 대환영😊😊

Taehyeon-Kim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

i-colours-u Oct 19, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

i-colours-u Oct 19, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

i-colours-u Oct 26, 2021 • edited Loading

Choose a reason for hiding this comment

i-colours-u Oct 19, 2021 • edited Loading

Choose a reason for hiding this comment

hyesuuou left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hellozo0 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jane1choi left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jumining left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

i-colours-u Oct 19, 2021 •

edited

Loading

i-colours-u Oct 19, 2021 •

edited

Loading

i-colours-u Oct 26, 2021 •

edited

Loading

i-colours-u Oct 19, 2021 •

edited

Loading