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#223] 이미지 선택 시 버튼 활성화되지 않는 버그 #234

Conversation

leemhyungyu
Copy link
Member

@leemhyungyu leemhyungyu commented Sep 4, 2024

이슈 #223

완료된 기능

  • ProfileImageUploadView WithPerceptionTracking 추가

전달사항

  • 전달받은 캐릭터 이미지를 통해서 1.0.5 release버전으로 테스트를 해본 결과 문제 없이 작동함.
  • 제보받은 디바이스의 화면을 확인해봤을 때 이미지 선택 후 imagePickerButton에 선택된 이미지가 보여지는 것을 봐서는 이미지 선택하는 로직쪽에는 문제가 없는 것 같음
private extension ProfileImageUploadView {
  func handleSelectedPhotos(_ newPhotos: [PhotosPickerItem]) {
    for newPhoto in newPhotos {
      newPhoto.loadTransferable(type: Data.self) { result in
        switch result {
        case .success(let data):
          if let data = data, let newImage = UIImage(data: data) {
            DispatchQueue.main.async {
              selectedImage = [newImage] // 빈 값
              let compressData = newImage.compressImageData() // nil로 처리
              store.send(.imageDidSelected(selectedImageData: compressData ?? .init()))
            }
          }
          
          // TODO: 이미지 로드 실패 처리
        case .failure(_):
          return
        }
      }
    }
    
    selectedItems.removeAll()
  }
}

버튼을 활성화시키는 부분은 Action의 imageDidSelected 을 통해 State의 isDisableDoneButton 의 값을 변경시켜 doneButton의 활성화여부를 바인딩해주고 있음.

각각 selectedImage에 빈 배열과 nil을 넣어도 문제없이 버튼이 활성화됨.
따라서, store 쪽에서 문제가 발생하는 것으로 판단했고 확인해보니깐 WithPerceptionTracking 관련해서 경고를 내뿜고 있었음

일단 ProfileImageUploadView body를 WithPerceptionTracking로 감싸서 view에서 store의 상태변경을 잘 추적하도록 함.

버그가 재연이 안되다보니 완벽하게 해결된지는 모르겠음...

@leemhyungyu leemhyungyu added this to the 1.0.6 milestone Sep 4, 2024
@leemhyungyu leemhyungyu self-assigned this Sep 4, 2024
@leemhyungyu leemhyungyu linked an issue Sep 4, 2024 that may be closed by this pull request
@leemhyungyu leemhyungyu changed the base branch from develop to release/1.0.6 September 4, 2024 12:28
@JongHoooon
Copy link
Member

선택한 사진의 상태는 변하는거같은데 disable 관련 상태는 반영이 안되는거같아 보여서 일단 WithPerceptionTracking 처리해서 배포하고 가능하다면 인준이형 지인한테 테스트를 한번 부탁드릴까?

@leemhyungyu
Copy link
Member Author

오케이! 일단 저거 아니면 딱히 문제 될 부분 없는거같으니깐 배포해도될 듯!

@leemhyungyu leemhyungyu force-pushed the fix/#233-이미지-선택-시-버튼이-활성화-되지-않는-버그 branch from c333e0b to 5384181 Compare September 4, 2024 15:08
@leemhyungyu leemhyungyu merged commit 9e5f416 into release/1.0.6 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이미지 선택 시 버튼이 활성화 되지 않는 버그
2 participants