Feat/#2 #3

heerucan · 2021-10-07T15:25:19Z

📌 관련 이슈

close #2

📌 변경 사항 및 이유

1차 과제 구현 완료입니당~!

📌 PR Point

private을 언제 써주는 건지 사실 잘 모르겠네요.
lazy var도 사실 좀 모르겠는데 항상 아리까리한 부분.
어떻게 하면 반복되는 코드들을 간결하게 줄일 수 있을까요?
특히, textField나 label 같은 것들... 계속 반복되는데 모르겠다.

📌 참고 사항

왜 이렇게 했는지 궁금하면 그냥 다 갈겨주세요.

i-colours-u

고생하셧슴니다

jane1choi

코드 엄청 깔끔하게 짜신 것 같고 변수명, 파일명 짓는 법에 대해서도 배워갑니다!

jane1choi · 2021-10-08T09:05:25Z

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

+            make.width.equalTo(80)
+            make.height.equalTo(50)
+        }
+    }


오토레이아웃까지 적용하셨네요,,짱이에요,,

2주차에 아마 스토리보드로 오토레이아웃 갈기는 법을 배울 것입니다릐..
스냅킷으로도 할 수 있지만 스보로도 충분히 할 수 있다!!

Snapkit 처음보는 용어인데 오토레이아웃 적용을 저렇게 할 수 있군용..코드로 적용하니까 수정하기 더 편해보여요!

jane1choi · 2021-10-08T09:10:04Z

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

+            signInButton.isUserInteractionEnabled = true
+            signInButton.backgroundColor = .mainBlue
+        }
+    }


trimmingCharacters는 뭘까요?? 엄청 깔끔하게 구현되네요,,

https://m.blog.naver.com/PostView.naver?isHttpsRedirect=true&blogId=traeumen927&logNo=221233934563
여기 이 링크에 좀 자세하게 설명이 나와있는데
여기 아래 사진에 나와있는 것처럼 white~ 머시기랑 같이 쓰면 문자열과 같이 있는 공백을 제거해줘용!!
그래서 공백을 입력해도 다음 버튼이 활성화되지 않게 하려고 써줬어여!

오오 저는 공백 생각도 못했는데..! 참고해서 저도 담번에 함 써먹어보겠습니다! 감사해용😊😊

저도 입력만 생각하고 공백 생각 못했는데 공백 생각하신 점 너무 좋아요. 놓쳤던 생각인데 알아갑니다!

jane1choi · 2021-10-08T09:12:23Z

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

+        configShow.baseBackgroundColor = .clear
+        configShow.imagePlacement = .leading
+        configShow.imagePadding = 10
+    }


이건 어떤 함수인가요??

이거는 iOS15에서 새로 나온 버튼과 관련된 함수인데 자세한 건 https://www.notion.so/SOPT-1-620e04ad885048968aa3b60a81bd5b35#3577ea3858774b5b9fb66939303f3d3f 여기를 참고해주세열

와앙 ConfigurationUpdateHandler 요거 잘쓰면 증맬 유용할 것 같아요!! 잘 참고하겠습니당 감사해요~!~!

jane1choi · 2021-10-08T09:15:31Z

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

+        } else {
+            pwTextField.isSecureTextEntry = true
+        }
+    }


너모 깔끔하네요,,짱이에요

하릥. 감사합니당.. 😵‍💫❤️

i-colours-u · 2021-10-08T16:05:54Z

1 )private을 언제 써주는 건지 사실 잘 모르겠네요.

--> 요건 해당 파일내에서 동작하는 메서드나 프로퍼티라면 전부 다 private로 처리해주는게 맞아요!!
외부에서 참조하거나 꺼내쓸때는 private가 아닌걸로 처리해주시면 되구..

2 ) lazy var도 사실 좀 모르겠는데 항상 아리까리한 부분.
--> 화면을 우리가 띄웠을 때 지금당장 필요하지 않은것들 있죠?! 그.. 예를 들어 버튼을 눌러야 나오는 "Picker View"라던지,
특정 행동을 해야 나오는 "View"라던지.. 지금당장 필요하지 않은 프로퍼티들에 대해서 이따가 호출할때 나와!! 라고 지정하는거에요.
그래서 화면이 뜨자마자 발생하는 UI나 쓰는 프로퍼티들에 대해서는 lazy를 사용해도되고? 안해도 상관 없습니다!!

3 ) 어떻게 하면 반복되는 코드들을 간결하게 줄일 수 있을까요?
특히, textField나 label 같은 것들... 계속 반복되는데 모르겠다.
--> 요것도 제가 항상 고민하는 부분인데..
큰 플젝을 하게 되면은, 디자이너분들이랑 항상 얘기해서 어느정도 프로덕트 System Design을 어떻게 설정할지 큰 틀 정도를 잡게될거에요!
공통되는 버튼은 이런 형태를 쓰고,, TextField는 요런걸 쓰고,, Label도 프로덕트 내에서 스타일이 정리가 되어있을거에요!! (title,subtitle, content 등등 해서 4,5개 내외로..)
그렇게 자주 쓰이는 UI 요소들에 대해서는 저는 그냥 class 로 따로 빼두는 편이에요! 스보를 쓰던, 코드로 쓰던 따로 클래스로 분리해놓으면 필요할때마다 꺼내쓰면 아무래도 일일이 지정하는 코드를 줄일수 있겠죠..?! 저는 그렇게 처리하는 편이에요!!

i-colours-u · 2021-10-08T16:09:34Z

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

+        guard let name = nameTextField.text,
+              let email = emailTextField.text,
+              let pw = pwTextField.text else {
+                  return
+              }


guard문 활용한거 좋네여 👍🏻

i-colours-u · 2021-10-08T16:14:08Z

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

+    func setTextField(placeholder: String, secure: Bool) {
+        self.placeholder = placeholder
+        self.isSecureTextEntry = secure


isSecureTextEntry까지 같이 해주는 섬세함 좋네요.... 감동받았습니다...

저는 저 위에 질의응답해주신 것 보고 진짜 감격의 도가니탕입니다.. 진진자라 미르스틴 그동안의 모든 의문점이 다 해소되는 답변..ㅠ 🙇‍♀️

i-colours-u · 2021-10-08T16:16:02Z

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

+
+    lazy var showButton = UIButton(configuration: configShow, primaryAction: nil).then {
+        $0.addTarget(self, action: #selector(touchupShowButton(_:)), for: .touchUpInside)
+        $0.configurationUpdateHandler = { btn in
+            var config = btn.configuration
+            config?.image = btn.isSelected ? UIImage(systemName: "checkmark.square.fill") : UIImage(systemName: "square")
+            btn.configuration = config


버튼에 대해서 configuration 가져와서 상태에 따라 이미지 처리하는거 좋네요!!! 나중에 iOS15정착되면 적극적으로 플젝 내에서 활용하면 좋을 것 같아요 !!!! 👍🏻👍🏻👍🏻

그니까요 이번에 처음 써봤는데 되게 좋은 기능인 거 같아열
긍데 갈기고보니 안에 var config = UIButton.configure.plain() ~ 요론식으로 넣어줄 수 있더라고요..;;ㅋ

YoonAh-dev

후릐님 LGTM❤️

YoonAh-dev · 2021-10-10T07:17:22Z

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

+        if sender.isSelected {
+            pwTextField.isSecureTextEntry = false
+        } else {
+            pwTextField.isSecureTextEntry = true


Suggested change

if sender.isSelected {

pwTextField.isSecureTextEntry = false

} else {

pwTextField.isSecureTextEntry = true

pwTextField.isSecureTextEntry = sender.isSelected? false : true

이렇게 써보는 건 어떨까요?

미르스틴 삼항연산자 알고 있는데 활용 못하는 나 제법 바오밥오나무같아요.

단순 Boolean 형태라면 요렇게도 작성할 수 있을 것 같습니다.

pwTextField.isSecureTextEntry = !sender.isSelected

@Taehyeon-Kim 오 바로 고쳐야겠습니다~!

YoonAh-dev · 2021-10-10T07:20:00Z

Sopt29th-Assignment/Podfile

+  pod 'SnapKit', '~> 5.0.0'
+  pod 'Then'
+  pod 'lottie-ios'
+  pod 'Moya', '~> 14.0'
+  pod 'Kingfisher', '~> 6.0'
+  pod 'SwiftLint'


뭐가 많네요...🤭

나중에 과제하다보면 다 추가해야 할 거 같아서 미리 추가해뒀습니다.😵‍💫😵‍💫

근데 왜 스위프트린트 추가했지..하하.. 쓸 거라고 생각했는데 안쓰네..

yangsubinn · 2021-10-10T11:32:59Z

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

+import UIKit
+
+extension UIViewController {
+    func hideKeyboard() {


손배릠 extension 너무 좋네여 쇽샥해가겠습니다~🏃‍♀️

저두 어딘가에서 쇽샥하였는데 감사함당❤️❤️

Taehyeon-Kim

LGTM 후릐 선배 ✨

Taehyeon-Kim · 2021-10-10T11:32:28Z

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

+        $0.addArrangedSubview(nameTextField)
+        $0.addArrangedSubview(emailTextField)
+        $0.addArrangedSubview(pwTextField)


이 부분도 stackview extension으로 빼서 사용할 수 있더라구요. UIView extension 만들어주신 것처럼 하셔도 좋을 것 같습니다👋

extension UIStackView { public func addArrangedSubviews(_ views: UIView...) { views.forEach(addArrangedSubview(_:)) } }

사실 태현쓰 코드 보러 갔다가 쇽샥 긁어왔다.. 좋은 Extension.. 입니다✨✨

Taehyeon-Kim · 2021-10-11T16:54:39Z

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

+        if sender.isSelected {
+            pwTextField.isSecureTextEntry = false
+        } else {
+            pwTextField.isSecureTextEntry = true


단순 Boolean 형태라면 요렇게도 작성할 수 있을 것 같습니다.

pwTextField.isSecureTextEntry = !sender.isSelected

Taehyeon-Kim · 2021-10-11T17:05:07Z

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

+    @objc func touchupSignupButton(_ sender: UIButton) {
+        let vc = CompleteVC()
+        vc.name = nameTextField.text
+        vc.modalPresentationStyle = .fullScreen


요건 그냥 여담인데 .fullScreen과 .overFullScreen에도 차이가 있네요!
사용해야 하는 순간을 적절하게 선택할 수 있을 것 같습니다.
https://stackoverflow.com/questions/58657826/benefit-of-using-fullscreen-presentation-style-in-compare-to-overfullscreen-pr

승찬님 PR에 정빈님이 남기신 코멘트보고 흥미로워서 찾아봤습니다~!

.fullScreen

프레젠테이션이 완료된 후 프레젠테이션 뷰 컨트롤러에 속한 뷰가 제거됩니다.

간단하게 말해서 모달 방식으로 화면 전환되었을 때, 이전 계층의 뷰가 없어진다는 것.

viewDidAppear를 호출한다는 것.

.overFullScreen

프레젠테이션이 끝날 때 프레젠테이션 아래의 뷰가 뷰 계층 구조에서 제거되지 않습니다.

간단하게 말해서 모달 방식으로 화면 전환되었을 때, 이전 계층의 뷰가 남아있다는 것

viewDidAppear를 호출하지 않는다는 것.

생각해보면 뷰를 불투명하게 올리지 않았을 때, 밑에 콘텐츠가 보이겠네요..?

@Taehyeon-Kim 그럼 지금 저 상황에서는 어떤 걸 써줘야 적합하나요?
보편적으로 사용자가 앱을 로그인을 하고 곧바로 그 전으로 돌아가는 상황(= dismiss 해주는 상황)은 거의 없으니까
.fullScreen이 맞는 건가요??

그나저나 친절한 설명.. 댕큐베리...감사..

Taehyeon-Kim · 2021-10-11T17:05:45Z

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

+        self.layer.borderColor = UIColor.lightGray.cgColor
+        self.layer.cornerRadius = 7
+        self.clipsToBounds = true
+        self.addLeftPadding()


궁금한데 right padding은 따로 안주시나요..? (이것 따지는 거 아님 주의..🥲 )

ㅋㅋㅋㅋㅋㅋㅋㅋ F 모먼트
세심걸이 아니었다.. 적용해주겠습니다..!

hyesuuou

손배야 머싯다. 도약했다 코드 아주 깔끔하다 LGTM❤️

hyesuuou · 2021-10-11T17:13:35Z

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

+        if name.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty ||
+            email.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty ||
+            pw.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
+            signInButton.isUserInteractionEnabled = false
+            signInButton.backgroundColor = .lightGray
+


우왕 새로운 거 배워갑니다~

hyesuuou · 2021-10-11T17:14:54Z

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

+        $0.addTarget(self, action: #selector(touchupShowButton(_:)), for: .touchUpInside)
+        $0.configurationUpdateHandler = { btn in
+            var config = btn.configuration
+            config?.image = btn.isSelected ? UIImage(systemName: "checkmark.square.fill") : UIImage(systemName: "square")


크으으으깔끔스

진짜 희루캔선배 코드 잘 짜 인정

감사합니다.. 진진자라 과찬의 말씀... 둘 다.. 과찬과찬.. 잠이 안와서 두 명한테 코리 갈기러 갈게~
어우 아래 @hyesuuou 길게 달아줬는데 reply 할 곳이 없어서 이렇게 달아보아.. 길게 설명해줬는데 이해 진자. 뻥 안치고 순도 100퍼센트 이해되었어. 진심.. 언제 private 써야 하는지 깨달아스!

hyesuuou · 2021-10-11T19:06:40Z

private 는 다른 클래스에서 접근하지 않는거 앞에 씁니다~
만약에 내가 class First { ... }을 만들고 이 안에 private var index: Int = 3
요렇게 프라이빗으로 뭥갈 선언했어

그리고 내가 class Second { ... }
를 만들었다
긍데 갑자기 요 Second라는 클래스 내부에서 First 클래스의 index를 7로 수정시켜주고 싶은거야..
그러면 보통 우리는 First().index = 7
이라고 해주려고 하겠지..? 근데 이러면 오류가 나셔 . 왜냐면 index는 First클래스 안에서 private으로 선언되었기 때문에 First클래스가 아닌 다른 외부 클래스에서는 여기에 접근 할수가 없기 때문이야..

그래서 사실 외부에서 직접 접근하지 않거나 보안상(?) 그러지 않아랴 하는 곳엔 private를 써줘야해!!

하나 예시로 내가 First 클래스의 index는 무조건 1-100사이의 수로 하기로 했어. 다른 수가 들어오면 오류가 나셔. 그러면 외부에서 200을 넣어줫읗 때 바로 반영되면 큰 문제가 생길수도 있잔하

그럴때 index를 private로 설정하고,

func setIndex (idx: Int) {
   if idx > 100, idx < 0 { return }
   else { index = idx }
}

First 클래스에 요런 함수를 만들어줘서

외부에서 수를 넣어줄 때 First().setIndex(idx: 200)요렇게 넣어주면 저 함수의 if문에 걸리게 되고, 이제 걸러주는 역할을 하게되는거지..

(폰으로 갈겨서 컴으로 보면 레이아웃 다깨지는거아닐지..)

그래서 결론은••
고냥 간단하게 다른클래스에서 접근할수 없게 하는거!! 라고 생각하면 좋을거가타
글고 전에 그냥 주워들은건데 객체지향에서는 클래스 안에 있는걸 외부에 안보이게 하는게 약간 중요포인트라고 하더라구.. 그래서 사실 외부에 굳이 보여줄 필요 없는거나 딱히 접근하지 않는거에는 다 private 해주는게 좋은거가터

jumining

오토레이아웃 배운 뒤에 보니까 코드가 더 잘보이네요! 과제 부분 함수도 깔끔하게 구현하신거 좋았고 마크구문 사용하는 것도 참고하겠습니다! 코드 좋았어요😀🍎

jumining · 2021-10-12T02:11:18Z

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

+extension UIViewController {
+    func hideKeyboard() {


Extension파일 따로 만들고 extension으로 넣은 함수들이 많은데 extension 사용한 이유가 있을까용?? 장점이 큰가요???

반복된 코드를 방지해주려고 써주는 것 같아요 저는!
지금 이 hideKeyboard()의 경우는 본래 같은 경우에는 키보드가 사라져야 하는 2개의 VC에 구현부와 함께 다 넣어줘야 되는데 이렇게 extension으로 빼줄 경우에는 hideKeyboard() 코드 한 줄만 VC의 viewDidLoad() 안에 호출해주면 되기 때문에 코드가 짧아지거든요!

jumining · 2021-10-12T02:18:48Z

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

+            make.width.equalTo(80)
+            make.height.equalTo(50)
+        }
+    }


Snapkit 처음보는 용어인데 오토레이아웃 적용을 저렇게 할 수 있군용..코드로 적용하니까 수정하기 더 편해보여요!

jumining · 2021-10-12T02:29:28Z

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

+    let logoLabel = UILabel().then {
+        $0.font = .boldSystemFont(ofSize: 50)
+        $0.textColor = .mainBlue
+        $0.text = "Google"
+    }
+
+    let loginLabel = UILabel().then {
+        $0.font = .boldSystemFont(ofSize: 35)
+        $0.textColor = .black
+        $0.text = "로그인"
+    }
+
+    let explainLabel = UILabel().then {
+        $0.font = .boldSystemFont(ofSize: 16)
+        $0.textColor = .lightGray
+        $0.numberOfLines = 0
+        $0.textAlignment = .center
+        $0.text = "YouTube도 이동하며 계속하세요. 앱 및 Safari에서도 Google 서비스에 로그인됩니다."
+    }
+
+    private lazy var fieldStackView = UIStackView().then {
+        $0.axis = .vertical
+        $0.alignment = .fill
+        $0.distribution = .fillEqually
+        $0.spacing = 20
+        $0.addArrangedSubview(nameTextField)
+        $0.addArrangedSubview(emailTextField)
+        $0.addArrangedSubview(pwTextField)
+    }
+
+    let nameTextField = UITextField().then {
+        $0.setTextField(placeholder: "이름을 입력해주세요", secure: false)
+        $0.addTarget(self, action: #selector(textFieldDidChange(textField:)), for: .editingChanged)
+    }
+
+    let emailTextField = UITextField().then {
+        $0.setTextField(placeholder: "이메일 또는 휴대전화", secure: false)
+        $0.addTarget(self, action: #selector(textFieldDidChange(textField:)), for: .editingChanged)
+    }
+
+    let pwTextField = UITextField().then {
+        $0.setTextField(placeholder: "비밀번호 입력", secure: true)
+        $0.addTarget(self, action: #selector(textFieldDidChange(textField:)), for: .editingChanged)
+    }
+
+    let signupButton = UIButton().then {
+        $0.setTitle("계정만들기", for: .normal)
+        $0.setTitleColor(.mainBlue, for: .normal)
+        $0.setTitleColor(.white, for: .highlighted)
+        $0.titleLabel?.font = .boldSystemFont(ofSize: 16)
+        $0.addTarget(self, action: #selector(touchupSignupButton(_:)), for: .touchUpInside)
+    }
+
+    let signInButton = UIButton().then {
+        $0.isUserInteractionEnabled = false
+        $0.setTitle("다음", for: .normal)
+        $0.setTitleColor(.white, for: .normal)
+        $0.titleLabel?.font = .boldSystemFont(ofSize: 18)
+        $0.backgroundColor = .lightGray
+        $0.layer.cornerRadius = 10
+        $0.addTarget(self, action: #selector(touchupSignInButton(_:)), for: .touchUpInside)
+    }


Then 라이브러리는 이렇게 적용하는거군용 viewDidload에 속성정의해야 적용되는 줄 알았는데 요런 방법도 있네요..

jumining · 2021-10-12T04:23:47Z

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

+            signInButton.isUserInteractionEnabled = true
+            signInButton.backgroundColor = .mainBlue
+        }
+    }


저도 입력만 생각하고 공백 생각 못했는데 공백 생각하신 점 너무 좋아요. 놓쳤던 생각인데 알아갑니다!

jumining · 2021-10-12T04:24:46Z

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

+    @objc func textFieldDidChange(textField: UITextField){
+        guard let name = nameTextField.text,
+              let email = emailTextField.text,
+              let pw = pwTextField.text else {
+                  return
+              }
+
+        if name.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty ||
+            email.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty ||
+            pw.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
+            signInButton.isUserInteractionEnabled = false
+            signInButton.backgroundColor = .lightGray
+
+        } else {
+            signInButton.isUserInteractionEnabled = true
+            signInButton.backgroundColor = .mainBlue
+        }
+    }


구현 못했던 도전과제부분인데, 코드 보면서 참고해서 공부할게요!😀

jumining · 2021-10-12T04:29:47Z

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

+    lazy var showButton = UIButton(configuration: configShow, primaryAction: nil).then {
+        $0.addTarget(self, action: #selector(touchupShowButton(_:)), for: .touchUpInside)
+        $0.configurationUpdateHandler = { btn in
+            var config = btn.configuration
+            config?.image = btn.isSelected ? UIImage(systemName: "checkmark.square.fill") : UIImage(systemName: "square")
+            btn.configuration = config


혹시 lazy var 쓴 부분 질문인데 위에 비슷한 형식인? 부분들은 let으로 쓴 것 같은데 lazy var 부분도 let을 사용하면 안되는건가용?

let이면 안되는 이유 : lazy는 호출되기 전까지 초기화되지 않아 값을 알 수 없는데
상수인 let 같은 경우는 초기화가 끝나기 전에 값을 가져야 하기 때문에 사용하면 안됩니당~!

heerucan added 14 commits October 5, 2021 03:07

➕[ADD] 과제 프로젝트 파일 생성(#2)

e956d25

➕[ADD] podfile 추가(#2)

61825bc

➕[ADD] UIView Extension 추가(#2)

847cd8c

➕[ADD] UITextField Extension 추가(#2)

478c534

✨[FEAT] LoginVC UI 구현(#2)

30ac352

✨[FEAT] SignUpVC UI 구현(#2)

8df74cf

✨[FEAT] 도전과제2 비밀번호 표시 기능 구현(#2)

0c90bb5

✨[FEAT] 도전과제1 버튼 활성화 기능 구현(#2)

a7e9039

✨[FEAT] CompleteVC UI 구현(#2)

1b4320d

✨[FEAT] 데이터 전달 및 화면전환 구현(#2)

793114c

✅[CHORE] 공백인 경우 예외처리 적용(#2)

1e6daf3

➕[ADD] Keyboard dismiss Extension 추가(#2)

f26a7ea

✅[CHORE] lazy var 삭제(#2)

7629329

📝[DOCS] update Readme

7a6bfe4

i-colours-u reviewed Oct 7, 2021

View reviewed changes

heerucan self-assigned this Oct 7, 2021

heerucan added the assignment label Oct 7, 2021

jane1choi reviewed Oct 8, 2021

View reviewed changes

i-colours-u reviewed Oct 8, 2021

View reviewed changes

✅[CHORE] configuration 코드 수정(#3)

98a7c3c

YoonAh-dev reviewed Oct 10, 2021

View reviewed changes

heerucan requested review from dlwns33, ezidayzi, hryeong66, hwangJi-dev and hyesuuou October 10, 2021 11:26

heerucan requested review from i-colours-u, jane1choi, jumining, ningpop, pcsoyeon, Taehyeon-Kim, yangsubinn and YoonAh-dev October 10, 2021 11:26

yangsubinn reviewed Oct 10, 2021

View reviewed changes

heerucan removed request for ningpop, hryeong66, pcsoyeon, hwangJi-dev, hyesuuou and ezidayzi October 10, 2021 16:54

heerucan added the 😊 huree 😊 label Oct 10, 2021

Taehyeon-Kim approved these changes Oct 11, 2021

View reviewed changes

hyesuuou approved these changes Oct 11, 2021

View reviewed changes

jumining reviewed Oct 12, 2021

View reviewed changes

heerucan added 3 commits October 14, 2021 02:42

♻️[[REFACTOR] 1차 과제 코드리뷰 반영 리팩토링(#3)

86f80c7

📝[DOCS] update Readme

95dd29f

🔀[MERGE] pull main branch

3901ecf

heerucan merged commit 7cf1bc6 into main Oct 13, 2021

heerucan deleted the feat/#2 branch October 13, 2021 17:54

heerucan added the give me a code review 👻 label Oct 18, 2021

Feat/#2 #3

Feat/#2 #3

Conversation

heerucan commented Oct 7, 2021 • edited Loading

📌 관련 이슈

📌 변경 사항 및 이유

📌 PR Point

📌 참고 사항

i-colours-u left a 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

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 commented Oct 8, 2021 • edited Loading

i-colours-u Oct 8, 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

YoonAh-dev 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

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

heerucan Oct 11, 2021 • edited Loading

Choose a reason for hiding this comment

Taehyeon-Kim Oct 11, 2021 • edited Loading

Choose a reason for hiding this comment

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

heerucan Oct 11, 2021 • edited Loading

Choose a reason for hiding this comment

hyesuuou commented Oct 11, 2021 • edited Loading

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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

heerucan commented Oct 7, 2021 •

edited

Loading

i-colours-u commented Oct 8, 2021 •

edited

Loading

i-colours-u Oct 8, 2021 •

edited

Loading

heerucan Oct 11, 2021 •

edited

Loading

Taehyeon-Kim Oct 11, 2021 •

edited

Loading

heerucan Oct 11, 2021 •

edited

Loading

hyesuuou commented Oct 11, 2021 •

edited

Loading