Byttar til ein oppdatert fork av openHtmlToPdf-biblioteket. #176
Conversation
|
@madsop-nav ser ut som om det er noen unit tester som feiler |
This comment was marked as duplicate.
This comment was marked as duplicate.
Usikker, blir nok ikkje før i August eg kan se på det, er i pappaperm😅 |
Skjønner! Kos deg! Har funne ut at mykje av støyen kjem frå denne PDFBox-bugen: https://issues.apache.org/jira/browse/PDFBOX-5743. Blir fiksa i neste patch-versjon, så for den delen er det berre å vente, verkar det som. Ser meir på resten. |
|
Oppdatering: Bytta font frå Source Sans Pro til Roboto-Regular etter tips frå Dag, og da køyrde testane grønt. Har ikkje klart å finne ut meir av korfor. |
Hm, tenker at det er ok å bytte font i unit testene. Men viss du har tid og får hjelp, kan derre se, om derre finner den underliggende årsaken til at dei feiler. Gjerne lag ein Issue på det, så kan derre følge den evt opp der |
|
@madsop-nav får du til å bytte font type, så kan vi få denne merget inn og den andre PRen? |
Kan for så vidt det, men eg er på ingen måte sikker på at det berre er i testane det vil feile. |
Ser den. Eg tenker vi prøver og ser. Det er ein lokal unit test, som kjører opp hele imaget, som du evt kan verifisere med: |
Har funne ut litt meir:
Vil dermed tippe at denne fonten gjer noko med bokstavkombinasjonen ft, kanskje noko a la dette, og så har ny versjon av pdfbox/openhtmltopdf gjort noko med genereringa av akkurat det. |
|
Til orientering: Eg har lagt denne litt på is, og heller tatt eit skritt oppover i verdikjeda, så no jobbar eg direkte på openhtml2pdf-prosjektet, for å først få på plass release-pipeline og sånt der, og i neste runde lage ein versjon av biblioteket som bruker pdfbox 2 innanfor rammene av den nye organisasjonen. |
Heilt gull 🏅 |
…vlege blir ikkje lenger oppdatert.
This reverts commit 853bdbb.
madsop-nav
force-pushed
the
oppdatert_openhtmltopdf
branch
from
6a4ace2 to
fb8e419
Compare
Den opphavlege blir ikkje lenger oppdatert, ref danfickle/openhtmltopdf#921.
Den nye har ein litt kuriøs groupid i Maven central, men det er denne forken: https://github.com/openhtmltopdf/openhtmltopdf , som er nyleg oppdatert og verkar ganske real. Det er den som er peika på som der dei gjer vidareutvikling også i tråden over.
Ref også https://nav-it.slack.com/archives/C60FFACN5/p1696243986018369
Ser dog at første commit etter flyttinga over til ny GitHub-organisasjon er å oppgradere PDFBox frå 2 til 3, så mogleg det trengs endringar også i denne kodebasen for å kunne gjera det?