[FEAT]139_회원가입, 소셜 로그인 구현 #162
Conversation
comeheredart
requested review from
inddoni,
hryeong66,
hwangJi-dev and
parkikbum
| case 200...299: return .success("회원가입 성공 되엇군 !!!!!!!!") | ||
| case 400...499: return .requestErr("리퀘스트에러") |
There was a problem hiding this comment.
킹받아요ㅋ ㅋ
자기소개하지마세요 ㅎ ㅎ
|
|
||
| let emailVerifyInputView = JoinInputView(title: "이메일 인증번호", | ||
| subTitle: "이메일로 보내드린 인증번호를 입력해주세요.", | ||
| placeholder: "ex)울랄라") |
There was a problem hiding this comment.
??????????
디자인팀의 예시임니다 ㅋ ㅋ 불만있으신지?
| if textField.text == "" { | ||
| isNicknamePassed = false | ||
| self.makeButtonsGray() | ||
| nicknameView.setOrangeTFLabelColorWithText(text: "닉네임을 작성해주세요.") |
There was a problem hiding this comment.
요거 그 hasText라고 텍스트 있는지 없는지 필드에서 체크해주는것이.. 있다고 하더라구요,,,
There was a problem hiding this comment.
요거 그 hasText라고 텍스트 있는지 없는지 필드에서 체크해주는것이.. 있다고 하더라구요,,,
댑악 다음에 반영해서 올리겟슴니다! ! !
| struct LoginJoinResponseDataModel : Codable { | ||
| let success: Bool | ||
| let msg: String | ||
| let data: UserInitialInfo | ||
| } | ||
|
|
||
| struct UserInitialInfo : Codable { | ||
| let email: String |
There was a problem hiding this comment.
Swift 컨벤션,, 띄어쓰기,, 초오콤 그시기 하네여,,,
| multipartFormData.append(imageData, withName: "image", fileName: "gg.jpeg", mimeType: "image/jpeg") | ||
| test.updateValue(imageData, forKey: "profileImage") |
| print(decodedData.success) | ||
| print(decodedData.data) | ||
|
|
There was a problem hiding this comment.
므엥,, return 뒤에 print 있으면 안찍히지 않을까요?
|
|
||
| default: return .networkFail | ||
| } | ||
| default: return .networkFail |
There was a problem hiding this comment.
나중에 개행 부탁드려요!
| default: return .networkFail | |
| default: | |
| return .networkFail |
| case 400: return .requestErr(decodedData.msg) | ||
| case 500: return .serverErr | ||
| default: return .networkFail | ||
| } |
| private func makeParameter(email : String) -> Parameters | ||
| { | ||
| return ["userEmail" : email] | ||
| } | ||
|
|
||
|
|
||
| func socialLogin(email : String, completion : @escaping (NetworkResult<Any>) -> Void) | ||
| { | ||
| let header : HTTPHeaders = ["Content-Type": "application/json"] |
There was a problem hiding this comment.
요기 대괄호 열리는 부분에 엔터쳐진 이유가 있나요,,? ㅠ 평소 컨벤션이랑 달라서 보기 어렵네요 ㅠㅠ
| self.isBig = isBig | ||
|
|
||
| if isBig { | ||
| self.setBackgroundImage(UIImage(named: "icSignupDisagreeBig"), for: .normal) | ||
| } else { | ||
| self.setBackgroundImage(UIImage(named: "icSignupDisagreeSmall"), for: .normal) |
There was a problem hiding this comment.
단순궁금증) if의 isBig의 파라미터로 들어온 isBig 맞을까요?
|
|
||
|
|
There was a problem hiding this comment.
전체적으로 함수 아랫 부분에 불필요한 개행이 많은 것 같아요 다음번에 한번에 조금 신경써주시면 좋을 것 같아요~~
| Agreed = !Agreed | ||
| if Agreed { | ||
| self.setBackgroundImage(UIImage(named: "icSignupAgreeBig"), for: .normal) | ||
| } else { | ||
| self.setBackgroundImage(UIImage(named: "icSignupDisagreeBig"), for: .normal) | ||
| } | ||
|
|
||
| case false: | ||
| Agreed = !Agreed | ||
| if Agreed { |
There was a problem hiding this comment.
Agreed를 true든, false든 스위치 시키는 것 같은데 이부분은 switch문 들어가기 전으로 빼도 될 것 같아요.
그리고 변수이름인데 왜 대문자로 되어있을까요? ㅠㅠ
! 말고 agreed.toggle() 이렇게 기본함수를 사용해도 좋을 것 같아요
…Ro/ChaRo-iOS into feat/139-make-Login-Join 🔨[FIX] : #139 충돌 해결
인도니씨 갬동입니다 .. 흑흑 |
📌 관련 이슈
#139
📌 변경 사항 및 이유
📌 PR Point
안녕하세요 오랜만입니다
아마 코드가 진짜 많고 ,,,
버그도 굉장히 많을건데 ,,, 천천히 봐주셧음 조켓다
📌 참고 사항
ㅠ ㅠ 제 무지성 코드를 용서해주세요