Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

PDF Viewer not loading fix#1000

Merged
dhaneshnm merged 3 commits into1.7.0-M3from
2969-pdf-not-loading
Dec 2, 2021
Merged

PDF Viewer not loading fix#1000
dhaneshnm merged 3 commits into1.7.0-M3from
2969-pdf-not-loading

Conversation

@nicwells
Copy link
Contributor

Fixes BlueBrain/nexus#2969

Description

PDF viewer was only working in dev. In dev/staging/production the pdf worker file could not be found.

Changed webpack config to copy pdfjs worker file to public directory. Specify this path for the worker.
Also specify ttf font file type separately to fix issue where path included public/public

How has this been tested?

Built for production and hosted locally to test. Confirmed the pdf worker file now loads and the pdf viewer works as expected.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added necessary unit and integration tests.
  • I have added screenshots (if applicable), in the comment section.

Also specify ttf font file type separately to fix issue where
path included public/public
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (1.7.0-M3@f41a08d). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head de729ae differs from pull request most recent head e41488b. Consider uploading reports for the commit e41488b to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##             1.7.0-M3    #1000   +/-   ##
===========================================
  Coverage            ?   66.00%           
===========================================
  Files               ?       21           
  Lines               ?      609           
  Branches            ?      137           
===========================================
  Hits                ?      402           
  Misses              ?      207           
  Partials            ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f41a08d...e41488b. Read the comment docs.

@dhaneshnm dhaneshnm merged commit 592785d into 1.7.0-M3 Dec 2, 2021
@dhaneshnm dhaneshnm deleted the 2969-pdf-not-loading branch December 2, 2021 09:28
dhaneshnm pushed a commit that referenced this pull request Dec 9, 2021
Also specify ttf font file type separately to fix issue where
path included public/public

Co-authored-by: Dhanesh Neela Mana <dhanesh.neelamana@epfl.ch>
dhaneshnm pushed a commit that referenced this pull request Mar 11, 2022
Also specify ttf font file type separately to fix issue where
path included public/public

Co-authored-by: Dhanesh Neela Mana <dhanesh.neelamana@epfl.ch>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants