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?

[VadimDez]

all require()s have been removed

[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.

[turbolego]

@VadimDez If i understand this correctly, unless an ECMAScript module bundle version of ng2-pdf-viewer is released, we have to add ng2-pdf-viewer to allowedCommonJsDependencies to ignore this warning? 🤔

Do you think an ECMAScript module bundle version of ng2-pdf-viewer will be available in the future? 🔭

[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.

[MikeMatusz]

Bump to remove stale.

[SKhalidQ]

Has anyone actually found a solution to this? Even if its temporary workaround?

[nischi]

https://github.com/mozilla/pdf.js/issues/10317

Still open :( Hope they will provide it.

[hal2814]

Not to sound like a broken record, but has there been any progress on this issue?

[korvised]

first create an ngcc.config.js file on the root => https://githubmemory.com/repo/VadimDez/ng2-pdf-viewer/issues/754

and then allowedCommonJsDependencies

"allowedCommonJsDependencies": [ "ng2-pdf-viewer", "pdfjs-dist" ]

it work for my project

[gultyayev]

Any progress? It seems like this type of module causes my initial bundle to have extra 500 Kb, while I'm using it only in lazy loaded modules so I'm expecting it not to be present in an initial bundle

[psarno]

Also hoping for any updates on this.

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