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

[Feat] #265 - fastlane & 유저 QA 진행 #267

Merged
merged 8 commits into from
Apr 15, 2024

Conversation

thingineeer
Copy link
Collaborator

@thingineeer thingineeer commented Apr 2, 2024

🌱 작업한 내용

  • fastlane appstore
  • 중복 닉네임 처리
  • 코스 업로드시 발생하는 이슈 수정
  • 2.0.3 버전 배포

🌱 PR Point

🍎 중복 코드 통일

    private func setKeyboardObservers() {
        NotificationCenter.default.addObserver(
            self,
            selector: #selector(keyboardWillShow),
            name: UIResponder.keyboardWillShowNotification,
            object: nil)
        NotificationCenter.default.addObserver(
            self,
            selector: #selector(keyboardWillHide),
            name: UIResponder.keyboardWillHideNotification,
            object: nil)
    }

키보드 관련 메서드가 두개였어서 위 메서드 하나로 통일 했습니다.

🍎 textView 사용자 입력 로직 변경

기존 로직을 수정해서, 제목 + 내용 을 입력해도 업로드 버튼이 활성화 되지 않는 이슈를 해결했습니다.

🍎 fastlane

fastlane release version:2.0.3 명령어로 심사까지 제출 할 수 있습니다.
하지만 메타데이터는 직접 파일을 수정해서 수정 해주셔야합니다.
ex) 업데이트 내용 , 사진 등등 ..

만약에 수정안할시 전 배포 버전이랑 동일하게 제출이 되어버립니다. !!

image

무려 딸깍 한 번에 심사까지 돼버립니다~~ 🫠

📸 스크린샷

중복 닉네임 처리 코스 업로드 로직 수정
코스업로드 중복닉네임

📮 관련 이슈

@thingineeer thingineeer added Setting 프로젝트 세팅, 라이브러리 설치 Fix 명진😼 labels Apr 2, 2024
@thingineeer thingineeer self-assigned this Apr 2, 2024
@thingineeer thingineeer requested a review from 513sojin April 3, 2024 04:14
Copy link
Collaborator

@513sojin 513sojin left a comment

Choose a reason for hiding this comment

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

중복 닉네임 처리 + 코스 업로드시 이슈 처리 확인했습니다!!
fastlane 쪽은 아는 게 없어서 코멘트를 못달았어요 .... .. .. 확인이 늦어서 미안합니다 고생하셨어요 🍀

@@ -175,11 +154,12 @@ extension CourseUploadVC {
scrollView.contentInset = contentInset
scrollView.scrollIndicatorInsets = contentInset

if activityTextView.isFirstResponder {
// scrollView 높이 설정
if courseTitleTextField.isFirstResponder || activityTextView.isFirstResponder {
Copy link
Collaborator

Choose a reason for hiding this comment

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

textView 누를때도 position 조정하신거 좋네요 !!!

Comment on lines +117 to +121
private func updateUploadButtonState() {
let isTitleNotEmpty = !(courseTitleTextField.text?.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty ?? true)
let isContentNotEmptyAndNotPlaceholder = !(activityTextView.text.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty || activityTextView.text == placeholder)
uploadButton.setEnabled(isTitleNotEmpty && isContentNotEmptyAndNotPlaceholder)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

업데이트 메소드 따로 빼둔거 확인했습니다~
textfield랑 textview 모두 개행 문자를 제거했을때 글자가 있고, 그 글자가 placeholder와 다른 경우 업로드 버튼 활성화로 이해했는데 맞을까요 ?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

맞아요!
코드를 조금 해석해보자면,

예를 들어 제목 같은 경우 text 가 nil 이면 !true를 리턴하여 최종적으로 false가 됩니다.
text가 있다면 텍스트 가 있기때문에 항상 !false 즉 최종적으로 true가 됩니다.

내용 같은 경우도 개행문자 제거했을 때 activityTextView가 비어있거나 || 플레이스 홀더 텍스트와 같은지 검사하여
필드가 비어있지 않고, 플레이스 홀더와 다를 경우 true 그렇지 않으면 false를 리턴합니다!

Comment on lines -110 to -113
@objc private func didNicknameReturn() {
guard let nickname = nickNameTextField.text else { return }
self.updateUserNickname(nickname: nickname)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

PR 상에서 잘 확인이 안돼서 그런데 이 함수는 원래는 어디에 쓰였던 함수인가요 ?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

닉네임을 치고, 완료 버튼을 누를 때 발생하는 이벤트 였습니다!
조금 코드를 변경하는 과정에서 중복 코드가 있어서 지운 것 같습니다!

@@ -184,6 +180,13 @@ extension NicknameEditorVC: UITextFieldDelegate {

extension NicknameEditorVC {
func updateUserNickname(nickname: String) {

guard nickname != self.currentNickname else {
Copy link
Collaborator

Choose a reason for hiding this comment

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

닉네임 동일 여부에 따른 처리 확인했습니다 ~

@thingineeer thingineeer changed the title [Feat] #265 - fastlane & 유저 QA 진헹 [Feat] #265 - fastlane & 유저 QA 진행 Apr 10, 2024
@thingineeer thingineeer merged commit c02e980 into Runnect:develop Apr 15, 2024
1 check passed
@thingineeer thingineeer deleted the #265-fastlane branch April 15, 2024 09:51
@thingineeer thingineeer added the UX label Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AppStore📱 Fix Setting 프로젝트 세팅, 라이브러리 설치 UX 명진😼
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] #265 - fastlane Release 배포 자동화 구현
2 participants