Conversation
There was a problem hiding this comment.
Sorry @ArgoZhang, your pull request is larger than the review limit of 150000 diff characters
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Bumps BootstrapBlazor.EmbedPDF to v10.0.6 and updates the bundled JS entrypoints to reference a new underlying embedpdf-*.js artifact (fixing #982 per PR description).
Changes:
- Bump NuGet/package version from 10.0.5 to 10.0.6
- Update
wwwrootJS entrypoints to import/export fromembedpdf-Ds41aXHo.js - Add a new re-export (
LockModeType) from the JS entry module
Reviewed changes
Copilot reviewed 3 out of 8 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/BootstrapBlazor.EmbedPDF/wwwroot/embedpdf.js | Updates public re-exports to point at new bundled embedpdf-*.js file and adds LockModeType export. |
| src/components/BootstrapBlazor.EmbedPDF/wwwroot/browser-BKLM0ThC-Bc3-ryZW.js | Updates internal import to the new embedpdf-*.js bundle. |
| src/components/BootstrapBlazor.EmbedPDF/BootstrapBlazor.EmbedPDF.csproj | Bumps package version to 10.0.6. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1 +1 @@ | |||
| export{V as AnnotationPlugin,W as AttachmentPlugin,X as BookmarkPlugin,Y as CapturePlugin,Z as CommandsPlugin,_ as DocumentManagerPlugin,$ as EmbedPdfContainer,a0 as ExportPlugin,a1 as FormPlugin,a2 as FullscreenPlugin,a3 as HistoryPlugin,a4 as I18nPlugin,a5 as InteractionManagerPlugin,a6 as PanPlugin,c as PdfAnnotationSubtype,a7 as PrintPlugin,a8 as RedactionMode,a9 as RedactionPlugin,aa as RenderPlugin,ab as RotatePlugin,R as Rotation,ac as ScrollPlugin,ad as ScrollStrategy,ae as SearchPlugin,af as SelectionPlugin,ag as SignatureMode,ah as SignaturePlugin,ai as SpreadMode,aj as SpreadPlugin,ak as ThumbnailPlugin,al as TilingPlugin,am as UIPlugin,an as ViewportPlugin,ao as ZoomMode,ap as ZoomPlugin,aq as createTheme,ar as darkTheme,as as default,at as deserializeEntries,au as ignore,av as lightTheme,aw as registerIcon,ax as registerIcons,ay as serializeEntries,az as version}from"./embedpdf-Buwm4uOT.js"; | |||
| export{V as AnnotationPlugin,W as AttachmentPlugin,X as BookmarkPlugin,Y as CapturePlugin,Z as CommandsPlugin,_ as DocumentManagerPlugin,$ as EmbedPdfContainer,a0 as ExportPlugin,a1 as FormPlugin,a2 as FullscreenPlugin,a3 as HistoryPlugin,a4 as I18nPlugin,a5 as InteractionManagerPlugin,a6 as LockModeType,a7 as PanPlugin,c as PdfAnnotationSubtype,a8 as PrintPlugin,a9 as RedactionMode,aa as RedactionPlugin,ab as RenderPlugin,ac as RotatePlugin,R as Rotation,ad as ScrollPlugin,ae as ScrollStrategy,af as SearchPlugin,ag as SelectionPlugin,ah as SignatureMode,ai as SignaturePlugin,aj as SpreadMode,ak as SpreadPlugin,al as ThumbnailPlugin,am as TilingPlugin,an as UIPlugin,ao as ViewportPlugin,ap as ZoomMode,aq as ZoomPlugin,ar as createTheme,as as darkTheme,at as default,au as deserializeEntries,av as ignore,aw as lightTheme,ax as registerIcon,ay as registerIcons,az as serializeEntries,aA as version}from"./embedpdf-Ds41aXHo.js"; | |||
There was a problem hiding this comment.
This entrypoint now re-exports from ./embedpdf-Ds41aXHo.js. If that new hashed bundle is not included in the published wwwroot (and the old embedpdf-Buwm4uOT.js is removed), consumers will hit a runtime 404/module-load failure. Ensure embedpdf-Ds41aXHo.js is added to the package output and that any static-web-assets manifest/publish include list is updated accordingly.
| export{V as AnnotationPlugin,W as AttachmentPlugin,X as BookmarkPlugin,Y as CapturePlugin,Z as CommandsPlugin,_ as DocumentManagerPlugin,$ as EmbedPdfContainer,a0 as ExportPlugin,a1 as FormPlugin,a2 as FullscreenPlugin,a3 as HistoryPlugin,a4 as I18nPlugin,a5 as InteractionManagerPlugin,a6 as LockModeType,a7 as PanPlugin,c as PdfAnnotationSubtype,a8 as PrintPlugin,a9 as RedactionMode,aa as RedactionPlugin,ab as RenderPlugin,ac as RotatePlugin,R as Rotation,ad as ScrollPlugin,ae as ScrollStrategy,af as SearchPlugin,ag as SelectionPlugin,ah as SignatureMode,ai as SignaturePlugin,aj as SpreadMode,ak as SpreadPlugin,al as ThumbnailPlugin,am as TilingPlugin,an as UIPlugin,ao as ViewportPlugin,ap as ZoomMode,aq as ZoomPlugin,ar as createTheme,as as darkTheme,at as default,au as deserializeEntries,av as ignore,aw as lightTheme,ax as registerIcon,ay as registerIcons,az as serializeEntries,aA as version}from"./embedpdf-Ds41aXHo.js"; | |
| export{V as AnnotationPlugin,W as AttachmentPlugin,X as BookmarkPlugin,Y as CapturePlugin,Z as CommandsPlugin,_ as DocumentManagerPlugin,$ as EmbedPdfContainer,a0 as ExportPlugin,a1 as FormPlugin,a2 as FullscreenPlugin,a3 as HistoryPlugin,a4 as I18nPlugin,a5 as InteractionManagerPlugin,a6 as LockModeType,a7 as PanPlugin,c as PdfAnnotationSubtype,a8 as PrintPlugin,a9 as RedactionMode,aa as RedactionPlugin,ab as RenderPlugin,ac as RotatePlugin,R as Rotation,ad as ScrollPlugin,ae as ScrollStrategy,af as SearchPlugin,ag as SelectionPlugin,ah as SignatureMode,ai as SignaturePlugin,aj as SpreadMode,ak as SpreadPlugin,al as ThumbnailPlugin,am as TilingPlugin,an as UIPlugin,ao as ViewportPlugin,ap as ZoomMode,aq as ZoomPlugin,ar as createTheme,as as darkTheme,at as default,au as deserializeEntries,av as ignore,aw as lightTheme,ax as registerIcon,ay as registerIcons,az as serializeEntries,aA as version}from"./embedpdf-Buwm4uOT.js"; |
Link issues
fixes #982
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge