skills icon indicating copy to clipboard operation
skills copied to clipboard

Make export not rely on proper image

Open Robin481 opened this issue 9 months ago • 1 comments

Sometimes images that were uploaded in Google Chrome broke the export. I think this is not an issue anymore, at least with new uploads. (sometimes an old image still breaks it)

Nevertheless, the export being reliant on the image being present seems unnecessary to me.

TODO:

  • Make export more safe against image problems (should still work with a nil or a broken image)
  • See if there are any other upload bugs (other file formats, browsers etc.)
  • Expert level: Display a small flash message that tells the user that there was a problem with the image and that it wasn't included in the export for that reason
  • Some cool tests maybe?

Robin481 avatar Apr 15 '25 06:04 Robin481

Stand 16.04.2025 Ich habe versucht das Problem lokal zu replizieren, jedoch ist mir dies nicht gelungen. Der upload erlaubt im frontend nur Bilddateien. Wenn man das HTML bearbeitet und trotzdem ein anderes Format höchlädt, dann lässt dies der Uploader nicht zu und es gibt einen processing error. Meiner meinung nach sollte der download momentan auch funktionieren, wenn eine Person kein Bild hat. Das einzige was ich schon angepasst habe, ist die condition, mit der überprüft wird, ob eine Person ein Bild hat.

RandomTannenbaum avatar Apr 16 '25 13:04 RandomTannenbaum