✨ [FEAT] 프로필 수정 뷰 수정사항 반영 #532

jane1choi · 2022-10-11T07:32:57Z

🍎 관련 이슈

closed #509

🍎 변경 사항 및 이유

프로필 수정 뷰의 UI 수정 사항을 반영했습니다.

🍎 PR Point

프로필 수정 뷰 바텀시트의 높이가 달라 present해주는 뷰에서 높이 지정을 할 수 있도록 코드를 수정했습니다.
프로필 수정 뷰의 경우 필요한 에셋이 없어 이벤트 로직 완성을 못했는데 디자인 파트에 요청해두었으니 추후에 이미지 전달까지 완료하도록 하겠습니다.
한줄소개 textView 추가 및 글자수 제한 기능을 구현했습니다.
제2전공 바텀시트의 경우 '미진입' 옵션이 있고 univID가 다를 경우 해당옵션의 majorID또한 다르기 때문에 EditProfileVC에서 학과리스트 요청 통신을 secondMajor filter를 걸어 통신을 한 번 더 해주었습니다.
커뮤니티 글쓰기 뷰에서 학과 무관 옵션의 경우 디폴트 MajorID 값을 박아두어서 유저가 선택안한 경우 해당 id값으로 요청이 가도록 했었는데 univID가 다를 경우 majorID또한 다르다는 걸 확인해서 싱글톤에 담아둔 id로 학교마다 다르게 구분되어 요청이 갈 수 있도록 코드를 수정했습니다.
EditProfileVC.swift파일 extension 순서 배치도 컨벤션에 맞게 바꿔놓아서 바뀐게 많아서 미리보기가 안뜰텐데 load diff눌러서 코드 확인해주세여...

+ 프로필 수정 뷰 UI는 거의 완료했는데 (에셋 나오지 않은거 + 진입시기의 경우 정빈이가 어떻게 리스트에 넣었는지 확인을 못해서 요부분빼고), 프로필 수정 요청 기능 구현의 경우 담당자가 원래 정빈이었어서 요거 이어서 맡을지 아니면 제가 계속할지? 논의가 필요할 것 같습니당! @dev-madilyn 확인 한번 해주세여~!

📸 ScreenShot

프로필 수정 뷰 바텀시트 + 한줄소개 글자수 제한

Simulator.Screen.Recording.-.iPhone.11.-.2022-10-11.at.16.24.54.mp4

본전공, 제2전공 바텀시트 비교

Simulator.Screen.Recording.-.iPhone.11.-.2022-10-11.at.16.25.14.mp4

바텀시트 수정완료

Simulator.Screen.Recording.-.iPhone.11.-.2022-10-15.at.16.08.44.mp4

dev-jungbin · 2022-10-11T14:41:15Z

프로필 수정 요청 기능 구현의 경우 담당자가 원래 정빈이었어서 요거 이어서 맡을지 아니면 제가 계속할지? 논의가 필요할 것 같습니당! @dev-madilyn 확인 한번 해주세여~!

@jane1choi "프로필 수정 요청 기능" 이라는 게 요청 request 말하는 건가? 이거랑 아마 저장버튼 활성화/비활성화 기능, 진입시기 안 될 것 같은데
진입시기는 SignUpMajorInfo뷰랑 똑같이 하면 되구... 저장버튼이랑 request 날리는 거는 모델 수정해야 해!

언니 이거 이어서 하는 거 혹시 부담됑 ??

dev-jungbin

확인이여 ~

dev-jungbin · 2022-10-11T14:47:45Z

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Mypage/ProfileSetting/VC/EditProfileVC.swift

    /// 제1, 제2전공 중복 선택 검사, 중복되지 않으면 true
    private func checkMajorDuplicate(firstTextField: UITextField, secondTextField: UITextField) -> Bool {
        return !(firstTextField.text == secondTextField.text)
    }
-}
-
-// MARK: - UI
-extension EditProfileVC {
-    private func configureUI() {
-        profileImgView.image = UIImage(named: "profileImage\(userInfo.profileImageID)")
-        nickNameTextField.placeholder = userInfo.nickname
-        isOnQuestionToggleBtn.isSelected = userInfo.isOnQuestion
-        firstMajorTextField.text = userInfo.firstMajorName
-        firstMajorStartTextField.text = userInfo.firstMajorStart
-        secondMajorTextField.text = userInfo.secondMajorName
-        secondMajorStartTextField.text = userInfo.secondMajorStart
-        checkSecondMajorStatus()
+
+    // 프로필 사진 변경 바텀시트를 present하는 메서드
+    private func presentChangeProfileImgVC() {
+        let slideVC = EditProfileImgModalVC()
+        slideVC.modalPresentationStyle = .custom
+        slideVC.transitioningDelegate = self
+        self.isPresentingHalfModal = false
+        self.present(slideVC, animated: true)
    }

-    private func changeLabelColor(isOK: Bool, label: UILabel) {
-        label.textColor = isOK ? .gray3 : .red
+    // HalfModalView를 present하는 메서드


여기 주석 /// 세 개로 통일해야 할 것 같아여 ~

dev-jungbin · 2022-10-11T14:48:04Z

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Mypage/ProfileSetting/VC/EditProfileVC.swift

+        secondMajorStartTextField.text = userInfo.secondMajorStart
+        checkSecondMajorStatus()
+
+        introTextView.rx.text.orEmpty


감사용 ~

This reverts commit 49e0181.

This reverts commit 06c1529.

This reverts commit 98ab550.

dev-jungbin · 2022-10-11T16:51:04Z

ㅎㅎ 작업한거 이 브랜치에 커밋 잘못 해서..,, 모두 revert 해주었습니당

hwangJi-dev

고생했어요~

…' into chore/#509-UI-Mypage-EditProfile

jane1choi · 2022-10-13T16:00:25Z

@hwangJi-dev @dev-madilyn
바텀시트 UI 적용완료 + 프로필 사진 선택 시 데이터 전달 까지 완료했습니다!
커밋은 687fac7 부터 봐주세요~

hwangJi-dev

리뷰 한번만 확인해주세요~

hwangJi-dev · 2022-10-13T17:25:15Z

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Mypage/ProfileSetting/VC/EditProfileImgModalVC.swift

+        case 1:
+            defaultSelectedBtn = profileImgBtn3
+        case 2:
+            defaultSelectedBtn = profileImgBtn5
+        case 3:
+            defaultSelectedBtn = profileImgBtn2
+        case 4:
+            defaultSelectedBtn = profileImgBtn4
+        case 5:
+            defaultSelectedBtn = profileImgBtn5
+        default:


요기 originalProfileImgID case number랑 profileImgBtn number랑 달라지는 이유가 무엇인가요?

hwangJi-dev · 2022-10-13T17:33:04Z

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Mypage/ProfileSetting/VC/EditProfileVC.swift

+            switch selectedImg {
+            case UIImage(named: "profileImage1"):
+                selectedProfileImgID = 1
+            case UIImage(named: "profileImage2"):
+                selectedProfileImgID = 2
+            case UIImage(named: "profileImage3"):
+                selectedProfileImgID = 3
+            case UIImage(named: "profileImage4"):
+                selectedProfileImgID = 4
+            case UIImage(named: "profileImage5"):
+                selectedProfileImgID = 5


ID 숫자값이랑 UIImage name 숫자값이랑 같으니까 delegate 전달할 때 UIImage의 에셋이름중 숫자값을 넘기는건 어떨까요?

dev-jungbin

확인!

dev-jungbin · 2022-10-14T04:44:15Z

NadoSunbae-iOS/NadoSunbae-iOS/Screen/Mypage/ProfileSetting/VC/EditProfileImgModalVC.swift

+        if sender != defaultSelectedBtn {
+            completeBtn.isActivated = true
+        } else {
+            completeBtn.isActivated = false
+        }


이부분 completeBtn.isActivated = sender != defaultSelectedBtn로 수정할 수 있을 것 같아여~!~

hwangJi-dev

확인했어요~ 고생했어!!

jane1choi added 10 commits October 9, 2022 01:18

➕ Add: 프로필 수정 버튼 추가 (#509)

7ec3b69

✅ Chore: 프로필 수정 뷰 기본 UI 수정사항 반영 (#509)

b9d3237

✨ Feat: 프로필 수정 한줄소개 글자수 표시 기능 구현 (#509)

ad050b5

✅ Chore: 바텀시트 높이 커스텀 가능하도록 코드 수정 (#509)

93767e1

✨ Feat: 프로필 이미지 변경 바텀시트 디폴트 UI 구성 (#509)

d0d5f80

✅ Chore: 제2전공 선택 시 미진입 옵션 처리 위한 코드 수정 (#509)

a5d3a04

✨ Feat: 바텀시트 호출 프로필 수정 뷰, 학과 리스트로 분기처리 (#509)

85c9a19

⏪️ Rename: @IBAction 네이밍 수정 (#509)

58e9fc2

✅ Chore: 컨벤션에 맞도록 함수 위치 조정 (#509)

80a12c8

✅ Chore: univID에 맞는 학과무관 majorID 보내도록 수정 (#509)

10606e6

jane1choi requested review from dev-jungbin and hwangJi-dev October 11, 2022 07:32

jane1choi self-assigned this Oct 11, 2022

dev-jungbin approved these changes Oct 11, 2022

View reviewed changes

dev-jungbin added 6 commits October 12, 2022 01:47

✅ Chore: 홈탭 엑세스토큰 갱신 처리 (#509)

98ab550

✅ Chore: 마이페이지탭 엑세스토큰 갱신 처리 (#509)

06c1529

✅ Chore: 알림탭 엑세스토큰 갱신 처리 (#509)

49e0181

Revert "✅ Chore: 알림탭 엑세스토큰 갱신 처리 (#509)"

2fe293f

This reverts commit 49e0181.

Revert "✅ Chore: 마이페이지탭 엑세스토큰 갱신 처리 (#509)"

ade5916

This reverts commit 06c1529.

Revert "✅ Chore: 홈탭 엑세스토큰 갱신 처리 (#509)"

733ed78

This reverts commit 98ab550.

hwangJi-dev approved these changes Oct 12, 2022

View reviewed changes

jane1choi added 7 commits October 13, 2022 16:20

➕ Add: 이미지 에셋 추가 (#509)

687fac7

✅ Chore: 함수 문서화 주석 수정 (#509)

9ca2dcf

🚚 Move: 이미지 에셋 폴더링 수정 (#509)

2eeef96

✅ Chore: 프로필 수정 뷰 디폴트 UI 수정 (#509)

82a1b0d

✅ Chore: 선택완료 버튼 조건 설정 (#509)

860aa81

✨ Feat: 버튼 액션 설정 및 데이터 전달 구현 (#509)

ad1362a

Merge remote-tracking branch 'origin/chore/#509-UI-Mypage-EditProfile…

c139720

…' into chore/#509-UI-Mypage-EditProfile

hwangJi-dev self-requested a review October 13, 2022 17:33

hwangJi-dev requested changes Oct 13, 2022

View reviewed changes

dev-jungbin approved these changes Oct 14, 2022

View reviewed changes

✅ Chore: 이미지 변경 및 리뷰반영한 코드 수정 (#509)

e9e7efe

hwangJi-dev approved these changes Oct 15, 2022

View reviewed changes

jane1choi merged commit 7759718 into develop Oct 17, 2022

jane1choi deleted the chore/#509-UI-Mypage-EditProfile branch October 17, 2022 07:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

✨ [FEAT] 프로필 수정 뷰 수정사항 반영 #532

✨ [FEAT] 프로필 수정 뷰 수정사항 반영 #532

jane1choi commented Oct 11, 2022 •

edited

Loading

dev-jungbin commented Oct 11, 2022

dev-jungbin left a comment

dev-jungbin Oct 11, 2022

dev-jungbin Oct 11, 2022

dev-jungbin commented Oct 11, 2022

hwangJi-dev left a comment

jane1choi commented Oct 13, 2022

hwangJi-dev left a comment

hwangJi-dev Oct 13, 2022

hwangJi-dev Oct 13, 2022

dev-jungbin left a comment

dev-jungbin Oct 14, 2022

hwangJi-dev left a comment

✨ [FEAT] 프로필 수정 뷰 수정사항 반영 #532

✨ [FEAT] 프로필 수정 뷰 수정사항 반영 #532

Conversation

jane1choi commented Oct 11, 2022 • edited Loading

🍎 관련 이슈

🍎 변경 사항 및 이유

🍎 PR Point

📸 ScreenShot

dev-jungbin commented Oct 11, 2022

dev-jungbin left a comment

Choose a reason for hiding this comment

dev-jungbin Oct 11, 2022

Choose a reason for hiding this comment

dev-jungbin Oct 11, 2022

Choose a reason for hiding this comment

dev-jungbin commented Oct 11, 2022

hwangJi-dev left a comment

Choose a reason for hiding this comment

jane1choi commented Oct 13, 2022

hwangJi-dev left a comment

Choose a reason for hiding this comment

hwangJi-dev Oct 13, 2022

Choose a reason for hiding this comment

hwangJi-dev Oct 13, 2022

Choose a reason for hiding this comment

dev-jungbin left a comment

Choose a reason for hiding this comment

dev-jungbin Oct 14, 2022

Choose a reason for hiding this comment

hwangJi-dev left a comment

Choose a reason for hiding this comment

jane1choi commented Oct 11, 2022 •

edited

Loading