Development #68641
toulouse-foederis: renvoyer l'url absolue des PDF des annonces
0%
Description
Dans la cellule annonce on a :
<a href="{{ passerelle_url }}{{ announce.pdf_url }}">PDF</a>
Mais alors on a un slash doublé entre le nom de domaine et l'url du endpoint.
Pour simplifier, on pourrait renvoyer l'url absolue (comme c'est fait dans le connecteur smart).
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0001-foederis-return-absolute-uri-of-pdf-announce-68641.patch 0001-foederis-return-absolute-uri-of-pdf-announce-68641.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
J'ai oublié de dire que passerelle refuse ces URLs avec le slash doublé (404)
(et que j'ai l'impression que c'est ça qui a introduit une régression dans le ticket lié)
Not Found: //toulouse-foederis/test/announce/5347110/pdf/
Mis à jour par Paul Marillonnet il y a plus d'un an
Euh, sans patcher, dans la cellule on ne peut pas simplement mettre
<a href="{{ announce.pdf_url }}">PDF</a>
?Mis à jour par Nicolas Roche il y a plus d'un an
Non, parce que le PDF doit être téléchargé sur passerelle.
Avec une adresse relative posée dans la cellule, on resterait sur combo.
(on peut tester ici : https://montoulouse.cutm-publik-preprod.nfrance.com/manage/pages/59/)
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Nicolas Roche a écrit :
Non, parce que le PDF doit être téléchargé sur passerelle.
Avec une adresse relative posée dans la cellule, on resterait sur combo.
(on peut tester ici : https://montoulouse.cutm-publik-preprod.nfrance.com/manage/pages/59/)
Ah oui ok j’avais mal compris la description du ticket. Fair enough.
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 959ce6c1973d7e115a35925580975d7444a89c56 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Sep 1 17:35:16 2022 +0200 foederis: return absolute uri of pdf announce (#68641
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
foederis: return absolute uri of pdf announce (#68641)