Skip to content

Conversation

@DavinAhn
Copy link
Collaborator

@DavinAhn DavinAhn commented Jan 2, 2023

작업 내용

  • PR과 기존 사용처에서의 검증 없이 일방적으로 a9f4fa0 를 커밋하였고, v2.2.0 때 잘못 포함되는 바람에 사이드 이펙트를 일으켰고 이를 롤백합니다.
  • GCD로 콜백을 제어할 수 있었으면 하는 제안을 수용한 것으로 메인 스레드와 백그라운드 스레드를 선택할 수 있는 형태로 재작업이 필요합니다.

@DavinAhn DavinAhn self-assigned this Jan 2, 2023
@Kyungtae-TEI
Copy link

GCD로 콜백을 제어할 수 있었으면 하는 제안

그런데 어떤 이슈로 인해 이 제안이 나온건지 혹시 기억하시나요???
evaluateJavaScript(_:completionHandler:) 의 콜백은 메인스레드에서 호출되는데 백그라운드로 바꾸려고 했던 이유가 무엇인지 궁금합니당

@DavinAhn DavinAhn merged commit 00f3059 into master Jan 3, 2023
@DavinAhn
Copy link
Collaborator Author

DavinAhn commented Jan 3, 2023

그런데 어떤 이슈로 인해 이 제안이 나온건지 혹시 기억하시나요???

아마 #7 이거요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants