MTES-MCT / Docurba

4 stars 5 forks source link

[Export PDF du PAC] REX #172

Closed CeliaVermicelli closed 11 months ago

CeliaVermicelli commented 1 year ago

@UngererFabien Je mets par ici les retours obtenus, qui rejoignent probablement ta conversation avec Laurence :

Bug taille des titres : Le titre de la section "Principe de la hiérarchie des normes" est aussi gros que le "a) Document dans l'évolution est engagé, etc." alors que c'est le corps du texte.

Capture d’écran 2023-10-27 à 18 15 59

Capture d’écran 2023-10-27 à 18 16 05

FYI @HermanceGauth

HermanceGauth commented 1 year ago

Le PAC commenté de la DDT38 (qui date d'il y a 1an et demi) MOD_PAC sec DREAL.odt.pdf

CeliaVermicelli commented 1 year ago

Hello @UngererFabien, je veux bien une update du bug du téléchargement du PAC du 22, [celui là](https://docurba.beta.gouv.fr/trames/pr Capture d’écran 2023-11-11 à 14 52 20 ojet-47e7dad0-9f7b-4cfb-b92d-50b64348cfc9). J'en parlais dans l'issue #61

Merci d'avance !

nyko28 commented 1 year ago

Hello @CeliaVermicelli, Pour le PAC du 22 qui bug à l'export PDF, c'est dû à une image hébergée sur sante.gouv.fr. Les CORS (politique de partage de ressource entre les sites) bloque le téléchargement de l'image pour l'inclure dans le PDF. J'ai édité le PAC pour remplacer l'image par la même image mais hébergée sur nos serveurs supabase directement. Elle devrait maintenant pouvoir l'exporter en PDF.

Pour le voir avec les dernières corrections sur ma version en local, je te mets l'export ici : PLU - Trédaniel - 2023_07_20.pdf

CeliaVermicelli commented 1 year ago

Génial @nyko28 !! Merci pour ces nouvelles, je la préviens et te tiens au courant si elle nous fait un retour.

Pour info, j'en ai profité pour retester les liens des titres du sommaire du .pdf qui devraient être cliquables et c'est assez aléatoire ; certaines fois ils sont cliquables et d'autres fois non, je n'arrive pas à trouver la constante. :/ J'ai remarqué que les sections mères sont cliquables quand il n'y a pas de sous-sections cochées pour qu'elles apparaissent dans le PAC. Et ensuite, sur le même PAC, une fois des sous-sections cochées sur la version en lignes, ces sous-sections sont bien cliquables mais alors leurs sections parentes ne le sont plus.. ? -> pas très digestes comme explications, n'hésite pas à me dire tes besoins.

CeliaVermicelli commented 1 year ago

Et @nyko28 une question au sujet de ces images et politique de partage de ressource entre les sites qui bloque l'export .pdf -> à ce stade on est capable de savoir comment contourner ce problème "automatiquement" ?

(ou bien avoir une alerte en interne quand un user clique plusieurs fois en peu de temps sur le bouton "PDF" pour lui venir en aide rapidement ?)

nyko28 commented 1 year ago

Oui il y a plusieurs solutions, je vais voir avec Fabien pour qu'on soit ok sur ce qui convient le mieux et je te tiens au courant

nyko28 commented 1 year ago

Update : le problème ne se posera pas pour les nouvelles images, car les utilisateurs ne peuvent pas insérer des images directement depuis une URL. Ils doivent la télécharger puis l'upload sur le PAC, ce qui n'entrainera pas de soucis.

L'image de sante.gouv.fr qui pose problème fait partie du PAC d'origine. Je vais faire un correctif pour que l'erreur ne se répète pas sur d'autres exports PDF.