Development #64347
Utiliser les legacy_urls dans un setting loader
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Lié à Support #57729: Migration d'instances de Publik : changement de nom de domaine ajouté
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-multitenant-add-legacy-urls-settings-loader-64347.patch 0001-multitenant-add-legacy-urls-settings-loader-64347.patch ajouté
- Sujet changé de Inclure les legacy_urls dans KNOWN_SERVICES à Utiliser les legacy_urls dans setting loader
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Changement de sujet du ticket, parce que bien plus pratique pour le cas d'usage évoqué d'avoir un nouveau setting loader qui crée un mapping, au lieu de juste ajouter ça à KNOWN_SERVICES.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Sujet changé de Utiliser les legacy_urls dans setting loader à Utiliser les legacy_urls dans un setting loader
Mis à jour par Thomas Noël il y a environ 2 ans
Optimisation minimale : tu pourrais faire un continue quand service.get('legacy_urls') est vide dans la boucle qui parcoure les services, sans continuer le reste (parce que ça sera le cas le plus fréquent).
Plus important : charger ce loader juste après KnownServices dans TENANT_SETTINGS_LOADERS ; en tout cas avant le SettingsJSON qui pourrait un jour contenir un update manuel de LEGACY_URLS_MAPPING. Qui sait...
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Fichier 0001-multitenant-add-legacy-urls-settings-loader-64347.patch 0001-multitenant-add-legacy-urls-settings-loader-64347.patch ajouté
Thomas Noël a écrit :
Optimisation minimale : tu pourrais faire un
continuequandservice.get('legacy_urls')est vide dans la boucle qui parcoure les services, sans continuer le reste (parce que ça sera le cas le plus fréquent).
J'ai pas compris, on est sur for legacy_url in service.get('legacy_urls', []):, si c'est vide on rentre pas dans la boucle et ça passe à l'itération suivant de la boucle du dessus.
Plus important : charger ce loader juste après KnownServices dans TENANT_SETTINGS_LOADERS ; en tout cas avant le SettingsJSON qui pourrait un jour contenir un update manuel de LEGACY_URLS_MAPPING. Qui sait...
J'ai pas tout compris non plus mais j'ai fait la modif.
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Fichier 0001-multitenant-add-legacy-urls-settings-loader-64347.patch 0001-multitenant-add-legacy-urls-settings-loader-64347.patch ajouté
Emmanuel Cazenave a écrit :
J'ai pas compris
Explication par oral, voilà ainsi modifié.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 69c7759bdbe9c9e3a0b94f97f3eb44f264858eb6
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Wed Apr 20 16:54:33 2022 +0200
multitenant: add legacy urls settings loader (#64347)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
multitenant: add legacy urls settings loader (#64347)