Skip to content

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 쪽은 아는 게 없어서 코멘트를 못달았어요 .... .. .. 확인이 늦어서 미안합니다 고생하셨어요 🍀


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.

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

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
@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