Ng2-pdf-viewer: Angular 10 issue -> optimization bailouts

[Seharad]

WARNING in /../node_modules/ng2-pdf-viewer/__ivy_ngcc__/fesm2015/ng2-pdf-viewer.js depends on pdfjs-dist/build/pdf. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

[JohnnyTMD]

same here

[PedroS5499]

same here, but put in angular.json like they said in official guide

[https://angular.io/guide/build#configuring-commonjs-dependencies]

"options": {
                        "allowedCommonJsDependencies": [",
                            "moment-business",
                            "ng2-pdf-viewer",
                            "pdfjs-dist/build/pdf",
                            "pdfjs-dist/web/pdf_viewer"
                        ],

[seopei]

this is just for ignoring the warnings. But the goal should be to get rid of the warnings in general.

[Seharad]

Ignoring errors is not the way to go.

[bekstn]

same here, any news?

[saithis]

pdfjs is not written with es modules, but they made a es module wrapper available that could be used in the meantime: https://www.npmjs.com/package/@bundled-es-modules/pdfjs-dist

for more info: https://github.com/mozilla/pdf.js/issues/10317

[VadimDez]

@saithis thanks for pointing the right issue
@bundled-es-modules/pdfjs-dist seems to be outdated

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[mrmokwa]

Can we re-open this one?

[PedroS5499]

@mrmokwa yes indeed

[PedroS5499]

any news?

[mememelm]

I hate this Warning :'(

[raichshlv]

Any updates?

[edusoccer1121]

Any updates on this?

[oussemazouaghi]

This warning is really annoying any pending fixes for this yet?

[imvikaskohli]

I m able to fix the issue by adding commonJsdependency property in the angular.json

angular.json

Inside Build command there is an option
"build": {
"builder": "@angular-devkit/build-angular:browser",
"options": {
"allowedCommonJsDependencies": ["ng2-pdf-viewer"],
}

[Seharad]

I m able to fix the issue by adding commonJsdependency property in the angular.json

angular.json

Inside Build command there is an option
"build": {
"builder": "@angular-devkit/build-angular:browser",
"options": {
"allowedCommonJsDependencies": ["ng2-pdf-viewer"],
}

this is not a fix, this is an ignore.

[stephanrauh]

@VadimDez When I tried to build you library from source, I ran into the same problem. Even worse: the require() statements work in the ng2-pdf-viewer demo project, but they failed to work after copying the dist folder to my test project.

Can we get rid of the require() statements? Angular's stopped supporting require.js. It'd be nice to use an import statement instead. Or we could load pdf.js and viewer.js directly. That'd allow for loading these big files lazily.

[nikhil-kapoor-quo]

any updates?