search

dataforgoodfr / batch7_rse

A search engine for French corporate societal and environnemental commitments and actions.
http://dataforgood.fr/batch7_rse/
MIT License
5 stars 2 forks source link

[Searbar & Detail View] Servir les PDFs des DPEFs #48

Closed CharlesGaydon closed 4 years ago

CharlesGaydon commented 4 years ago

Il faut que les PDFs soient téléchargeables. Cette discussion sur stack évoquent différentes approches. En gros deux familles:

  1. Servir les fichiers en y accédant par leur chemin local.
  2. Servir les fichiers sur un serveur apache nginx séparé du site, et ensuite utiliser les urls ainsi générées pour servir les fichiers considérés.

Cette deuxième approche est évidemment la plus propre. Elle rejoint l'issue https://github.com/dataforgoodfr/batch7_rse/issues/2 sur le fait de donner accès en téléchargement à la DB complète (en gros: il nous faudra bien ce genre de serveur à un moment).

Actuellement les DPEFs sont stockés dans batch7_rse\webapp\data\model\DPEFs, et lorsqu'ils sont ajoutés dans la BD ils sont aussi déplacé dans batch7_rse\webapp\data\polls\models\dpef. De là ils pourraient être servis directementpar Django (approche 1), en passant par le champs file_object du modèle DPEF. Ca ne semble pas trop compliqué de créer une vue qui permet d'ouvrir un pdf au sein de la page actuelle. Ce thread stack offre des solutions qui ont l'air assez simple pour télécharger ou afficher le pdf directement en accédant à l'objet de la BD.

CharlesGaydon commented 4 years ago

@Renaud-D-Harreville Tu avais fait quelques tests il me semble, est-ce qu'ils étaient plutôt suivant la méthode 1 (cf. deuxième thread stack sur le sujet) ? :)

CharlesGaydon commented 4 years ago

https://stackoverflow.com/a/11780055

CharlesGaydon commented 4 years ago

@Renaud-D-Harreville Pour info, première étape passée avec succès ;) image

CharlesGaydon commented 4 years ago

image

CharlesGaydon commented 4 years ago

Merged into branch pg_migration in https://github.com/dataforgoodfr/batch7_rse/pull/51