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: Explicitly import pdf worker entry to register global worker object #82

Merged
merged 1 commit into from
Dec 27, 2020

Conversation

fufler
Copy link
Contributor

@fufler fufler commented Dec 24, 2020

Latest pdfvuer version does not contain pdf worker. Thus demo page served by npm run serve does not work due to following errors:

app.worker.js:1 Uncaught SyntaxError: Unexpected token '<'
Uncaught (in promise) Error: Setting up fake worker failed: "Cannot read property 'WorkerMessageHandler' of undefined".

Describe the changes you have made in this PR -

This PR simply adds worker entry import to make worker globally accessible via window object.
Solved issue is probably related to #80.

Have you updated the readme?

No

Screenshots of the changes (If any) -

N/A

@fufler fufler force-pushed the explicitly-import-worker branch 2 times, most recently from f5759fa to 94bac72 Compare December 24, 2020 20:44
@fufler fufler changed the title Explicitly import pdf worker entry to register global worker object fix: Explicitly import pdf worker entry to register global worker object Dec 24, 2020
@arkokoley arkokoley merged commit dd59265 into arkokoley:master Dec 27, 2020
@arkokoley
Copy link
Owner

Published as part of v1.7.5

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.

2 participants