Development #34212
préremplissage à la création de service
Début:
21 juin 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Quand on crée un service via la page /sites/, le champs url de base est actuellement prérempli avec le domaine example.net. Ce ticket pour avoir le nom du domaine courrant.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Christophe Siraut il y a presque 5 ans
- Fichier 0001-environment-use-current-domain-name-for-services-cre.patch 0001-environment-use-current-domain-name-for-services-cre.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 5 ans
- Utiliser django.utils.six pour avoir un seul import pour urlparse.
- create_base_url(hostname, service_id) est déjà peu lisible là et je vois déjà la chose s'aggraver, l'accompagner de tests.
- comme SERVICE_URL_TEMPLATE n'est plus utilisé nulle part, le retirer (de settings et d'une autre référence inutile dans le fichier).
- je ne pense pas qu'utiliser service_id soit opportun, on a maintenant service_slug, qui correspond davantage à ce qu'on voudra dans un nom de domaine, mais idéalement il faudrait une variante supplémentaire, parce qu'on ne veut ni authentic.whatever ni idp.whatever, on veut connexion.whatever.
Mis à jour par Christophe Siraut il y a presque 5 ans
J'ai posté ma branche dans jenkins avec les corrections demandées. Reste la dernière remarque en suspens, avoir quelque part un sous-nom-de-domaine par défaut. Pourquoi ne pas modifier Extra.service_default_slug ?
Mis à jour par Christophe Siraut il y a presque 5 ans
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de En cours à Résolu (à déployer)
commit 6583fa518f7ea03548a25eaba553c2887890e288 Author: Christophe Siraut <csiraut@entrouvert.com> Date: Fri Jun 21 11:24:30 2019 +0200 environment: use current domain name for services creation form (#34212)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Lié à Bug #35368: Utiliser django.utils.six. ajouté
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
environment: use current domain name for services creation form (#34212)