-
Notifications
You must be signed in to change notification settings - Fork 4
[react-pdf] Promise.withResolvers 추가 #208
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
Conversation
✅ Changeset detectedLatest commit: d9c7c06
If no version change is needed, please add The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
|
|
여기 배포 세팅안했던거같은데 |
NPM Packages📦 @naverpay/react-pdfTotal Sizes: 22 kB Total Changes: +69 B (0%)
ℹ️ View Unchanged Files
powered by: naverpay size-action |
|
바로 리얼배포 들어가보겠습니다 |
2-one-week
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
상남자식 배포
| @@ -1,3 +1,7 @@ | |||
| // pdfjs-dist에서 Promise.withResolvers를 사용하므로 polyfill 주입을 위해 호출 | |||
| // eslint-disable-next-line @typescript-eslint/no-unused-expressions | |||
| Promise.withResolvers | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// polyfill 바닐라로 구현해서 넣기
if (typeof Promise.withResolvers !== 'function') {
Promise.withResolvers = function <T>() {
let resolve: (value: T | PromiseLike<T>) => void
let reject: (reason?: unknown) => void
const promise = new Promise<T>((res, rej) => {
resolve = res
reject = rej
})
return { promise, resolve: resolve!, reject: reject! }
}
}import 'core-js/actual/promise/with-resolvers'https://app.unpkg.com/[email protected]/files/actual/promise/with-resolvers.js
둘중 하나가 맞을듯..
Related Issue
Describe your changes
Promise.withResolvers 폴리필 주입을 위한 코드 추가
ios특정버전 미만에서(16.6.1 버전이상에서는 정상동작 확인) react-pdf 에러가 발생하는 문제의 원인으로 추정중. (테스트 필요)
Request