Fix [#65] 세부사항 수정

[Heyjooo]

👻 PULL REQUEST

💻 작업한 내용

• 마이페이지 스크롤 네비게이션 이슈 해결
• 회원가입 약관동의 분기처리 이슈 해결

💡 참고사항

• 소셜로그인을 하였는데 회원가입 완료를 하지 않은 유저를 분기처리하였습니다.
  

  Don-tBe-iOS/DontBe-iOS/DontBe-iOS/Presentation/Login/ViewModels/LoginViewModel.swift

  Lines 63 to 76 in 538e806

  	Task {
  	do {
  	let isNewUser = try await self.getSocialLoginAPI(accessToken: accessToken)?.data?.isNewUser ?? false
  	let nickname = try await self.getSocialLoginAPI(accessToken: accessToken)?.data?.nickName ?? ""
  	if !isNewUser && !nickname.isEmpty {
  	self.userInfoPublisher.send(false)
  	} else {
  	self.userInfoPublisher.send(true)
  	}
  	} catch {
  	print(error)
  	}
  	}

📸 스크린샷

기능	스크린샷
마이페이지 네비게이션바

📟 관련 이슈

• Resolved: [Fix] 세부사항 수정 #65

[github-actions]

주어진 코드에 대한 간단한 코드 리뷰입니다.

변경된 부분:

• extension UIViewController에서 statusBarHeight라는 계산된 프로퍼티가 추가되었습니다. 이 프로퍼티는 상태 표시줄의 높이를 반환합니다.
• UINavigationController extension이 삭제되었습니다.

버그 위험 및 개선 제안:

• 주어진 코드 패치에는 큰 문제가 보이지 않습니다. 단, 사용자 인터페이스와 관련하여 어떤 기능을 달성하려는지 명확하게 이해할 필요가 있습니다. 즉, 코드가 속하는 프로젝트의 전반적인 구조와 목적을 파악한 다음에야 더 정확한 리뷰를 할 수 있습니다.

일반적으로 코드 리뷰에는 다음과 같은 몇 가지 체크 포인트가 있을 수 있습니다:

1. 이름 짓기: 변수, 함수 및 클래스의 이름이 명확하고 일관성 있으며 의미있는지 확인합니다.
2. 가독성: 코드의 가독성을 향상시키기 위해 주석, 들여쓰기, 공백 등을 확인합니다.
3. 성능 및 메모리 관리: 코드가 효율적이고 자원을 효과적으로 사용하는지 확인합니다.
4. 잠재적인 오류: 코드에 잠재적인 버그가 있는지 확인합니다.

다만, 주어진 코드 패치가 개별 프로젝트의 일부분이기 때문에 상황에 따라 다른 리뷰 포인트가 필요할 수 있습니다.

[github-actions]

주어진 코드 패치의 간단한 코드 리뷰를 도와드리겠습니다. 버그 위험성과/또는 개선 제안 사항을 언급하겠습니다:

• UIGestureRecognizerDelegate 프로토콜을 추가하여 MyPageAccountInfoViewController가 제스처 인식자(delegate)를 처리할 수 있도록 했습니다.
• 주석인 // MARK: - Properties은 현재 존재하지 않아 추가되어야 합니다.
• 다른 부분에 대한 리뷰 또는 수정이 없으므로 해당 부분에서는 문제가 없습니다.

지금까지 코드 리뷰였습니다. 추가적인 정보나 질문이 있다면 알려주세요!

[yeonsu0-0]

와우 ~~! 고생했습니다👻🔥

[boogios]

P3
디테일 수정 고생하셨습니다!! 자잘자잘한 오류들 잘 잡아주셨네요!!