Development #29435
liens relatifs dans la navigation PWA
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
03 janvier 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Les entrées de la navigation PWA définies pour pointer vers une page locale pointent vers la page sans préciser de nom de domaine; une fois présent dans un template exploité par authentic/wcs/whatever, ça donne un lien qui ne marche pas :/
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-pwa-add-site-base-URL-for-local-navigation-entries-2.patch 0001-pwa-add-site-base-URL-for-local-navigation-entries-2.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce n'est pas super joli mais les alternatives auxquelles j'ai pu penser l'étaient encore moins.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Peut-être l'occasion de dire que url doit être absolue ; genre :
- url = models.CharField(verbose_name=_('External URL'), max_length=200, blank=True)
+ url = models.CharField(verbose_name=_('External URL (can be a template, must be absolute)'), max_length=200, blank=True)
Ok je suis pas bon en anglais, arrêtez de rire.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Il est écrit "External" du coup je considère que les gens comprendront que c'est fait pour pointer vers un site externe. (et j'ai la flemme d'adapter les migrations, peut-être, aussi).
commit 56139bed1f363cf338136a0d084909e260dfdc40
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jan 3 21:11:54 2019 +0100
pwa: add site base URL for local navigation entries (#29435)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Thomas Noël il y a plus de 5 ans
C'est fou ça, j'avais même pas vu le "External"... impec'.
pwa: add site base URL for local navigation entries (#29435)