Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] QA 반영 #359

Merged
merged 8 commits into from
Sep 4, 2024
Merged

[Fix] QA 반영 #359

merged 8 commits into from
Sep 4, 2024

Conversation

hooni0918
Copy link
Member

@hooni0918 hooni0918 commented Sep 2, 2024

🔗 연결된 이슈

📄 작업 내용

  • 온보딩 백버튼
  • 온보딩 이미지 교체시 킹피셔 캐시삭제
  • 서버측과 레벨명 분기
  • 프로필 설정화면진입시 탭바 삭제
  • 기본 프로필로 설정 텍스트
  • 테스트 로그인 삭제

👀 기타 더 이야기해볼 점

테스트 로그인은 배포를 위해 삭제되엇습니다
qa 처리 완료햇습니다
underline새로운 메서드를 추가햇습니다

@hooni0918 hooni0918 added ✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면 labels Sep 2, 2024
@hooni0918 hooni0918 self-assigned this Sep 2, 2024
@hooni0918 hooni0918 linked an issue Sep 2, 2024 that may be closed by this pull request
5 tasks
Copy link
Member

@mmaybei mmaybei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 👍🏻

@@ -35,6 +35,8 @@ class NicknameViewController: BaseViewController {
setupTextField()
setupTapGesture()
setupNavigationBarTitle(with: "닉네임 설정")
navigationItem.leftBarButtonItem = nil

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기 빈 줄 삭제 부탁드려요!

@@ -136,6 +138,7 @@ class NicknameViewController: BaseViewController {
.disposed(by: disposeBag)
view.addGestureRecognizer(tapGesture)
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

빈 줄 삭제!

Copy link
Contributor

@JinUng41 JinUng41 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다.

Comment on lines -65 to +52
}
print("uploadProfileImage 함수 호출됨")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 정도의 print문은 삭제해도 좋을 것 같습니다.

Comment on lines +79 to +80
print("프로필 이미지 업로드 성공")

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

성공했을 때의 print문이 필요할까요?

Comment on lines +32 to +50
func addUnderlineWithMyPage(
textColor: UIColor,
underlineColor: UIColor = .gray2,
spacing: CGFloat = 12
) {
let font = UIFont.pretendard(.body05)
let underlineStyle = NSUnderlineStyle.single

let attributes: [NSAttributedString.Key: Any] = [
.font: font,
.foregroundColor: textColor,
.underlineStyle: underlineStyle.rawValue,
.underlineColor: underlineColor,
.baselineOffset: spacing / 2
]

let attributedString = NSMutableAttributedString(string: self.titleLabel?.text ?? "", attributes: attributes)
setAttributedTitle(attributedString, for: .normal)
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member

@youz2me youz2me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍

@hooni0918 hooni0918 merged commit 084491e into suyeon Sep 4, 2024
@hooni0918 hooni0918 deleted the fix/#352-jihoonQaSolve branch September 4, 2024 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 구현시 사용 🛠️ fix 버그나 오류 해결시 사용 🧡 JiHoon 쌈뽕한 플러팅 꿀팁을 듣고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] QA 반영
4 participants