#24-PDF-Anzeigen #41

Merged
1924466 merged 3 commits from #24-PDF-Anzeigen into main 2025-05-30 07:36:32 +02:00

Added PDFviewr as component and extractedResults as page that uses PDFViewer.

Added PDFviewr as component and extractedResults as page that uses PDFViewer.
1924466 was assigned by 3019483 2025-05-27 16:03:32 +02:00
3019483 added 1 commit 2025-05-27 16:03:33 +02:00
3019483 changed title from #24-PDF Anzeigen to #24-PDF-Anzeigen 2025-05-27 16:03:44 +02:00
1924466 reviewed 2025-05-28 15:48:05 +02:00
1924466 left a comment
Collaborator
  • Es wird ein PDF-Viewer verwendet (z.B. react-pdf).

  • Die Datei ist komplett durchblätterbar, Seite für Seite.

  • Der PDF-Viewer ist sichtbar und zeigt die hochgeladene Datei korrekt an.
    verbesserungsvorschläge:

  • example.pdf scheint als Beispiel zum Testen. Wird dies noch gebraucht oder kann sie entfernt werden?

  • brauchen wir eine Ladeanzeige bei langsamen PDFs abload? habe es nicht gefunden.

  • brauchen wir Fehlerbehandlung wenn Laden von PDFs fehlt ?

- [x] Es wird ein PDF-Viewer verwendet (z.B. react-pdf). - [x] Die Datei ist komplett durchblätterbar, Seite für Seite. - [x] Der PDF-Viewer ist sichtbar und zeigt die hochgeladene Datei korrekt an. verbesserungsvorschläge: - example.pdf scheint als Beispiel zum Testen. Wird dies noch gebraucht oder kann sie entfernt werden? - brauchen wir eine Ladeanzeige bei langsamen PDFs abload? habe es nicht gefunden. - brauchen wir Fehlerbehandlung wenn Laden von PDFs fehlt ?
3019483 added 1 commit 2025-05-29 09:19:07 +02:00
3019483 added 1 commit 2025-05-29 09:30:07 +02:00
Poster
Owner
  • example.pdf scheint als Beispiel zum Testen. Wird dies noch gebraucht oder kann sie entfernt werden?

Ich habe es drin gelassen, weil andere Tickets davon auch profitieren (z.B. pdf highlighting). Sobald das Frontend mit dem Backend verbunden ist, sollten wir es raus nehmen. Ich würde es für die Zeit dort lassen.

  • brauchen wir eine Ladeanzeige bei langsamen PDFs abload? habe es nicht gefunden.

Ist bei dem react-PDF packet mit drin. Du kannst in den Dev tools die Network Geschwindigkeit auf 3G setzten und dann siehst du es. (siehe screenshot)

  • brauchen wir Fehlerbehandlung wenn Laden von PDFs fehlt ?
    done
- example.pdf scheint als Beispiel zum Testen. Wird dies noch gebraucht oder kann sie entfernt werden? Ich habe es drin gelassen, weil andere Tickets davon auch profitieren (z.B. pdf highlighting). Sobald das Frontend mit dem Backend verbunden ist, sollten wir es raus nehmen. Ich würde es für die Zeit dort lassen. - brauchen wir eine Ladeanzeige bei langsamen PDFs abload? habe es nicht gefunden. Ist bei dem react-PDF packet mit drin. Du kannst in den Dev tools die Network Geschwindigkeit auf 3G setzten und dann siehst du es. (siehe screenshot) - brauchen wir Fehlerbehandlung wenn Laden von PDFs fehlt ? done ✅
1924466 merged commit 416c2ceefd into main 2025-05-30 07:36:32 +02:00
1924466 deleted branch #24-PDF-Anzeigen 2025-05-30 07:36:33 +02:00
Sign in to join this conversation.
There is no content yet.