Development #63621
Faire apparaitre des infos tirées de legacy_urls en lecture seule sur /sites
0%
Description
Genre uniquement :
- la date de changement d'url
- l'ancien url de base (laisser tomber provisionning url etc)
Fichiers
Révisions associées
environment: show legacy urls (#63621)
Historique
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier Screenshot 2022-04-19 at 16-48-41 Système.png Screenshot 2022-04-19 at 16-48-41 Système.png ajouté
- Fichier 0001-environment-show-legacy-urls-63621.patch 0001-environment-show-legacy-urls-63621.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 environ 2 ans
J'ajouterais un mot pour dire à quoi correspond la date (sans lire le code je dirais "cette adresse jusqu'à cette date" mais c'est mieux de ne pas devoir deviner).
url → URL, urls → URLs.
legacy_url['datetime'] = datetime.datetime.fromtimestamp(float(legacy_url['timestamp']))
Il y a sans doute à reprendre un peu plus haut, sur le patch initial qui faisait timestamp = datetime.datetime.now()
, ça serait sans doute mieux d'y avoir utcnow() et de créer une datetime tz-aware ici.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-environment-store-UTC-timestamp-when-renaming-a-serv.patch 0001-environment-store-UTC-timestamp-when-renaming-a-serv.patch ajouté
- Fichier 0002-environment-show-legacy-urls-63621.patch 0002-environment-show-legacy-urls-63621.patch ajouté
Voilà.
(je ne me lance pas dans une migration des timestamp existants)
Mis à jour par Frédéric Péters il y a environ 2 ans
Ok on passe utcnow mais on change aussi de méthode pour taper un timestamp et doute intégral si jamais ça doublait l'écart par rapport à UTC ?
Désolé si ça semble un revirement mais pour la lisibilité, autre suggestion, ne pas enregistrer de timestamp mais la chaine formatée, '%Y-%m-%d %H:%M:%S'.
→ datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S') et datetime.datetime.strptime(timestamp, '%Y-%m-%d %H:%M:%S')
?
(totalement ok pour l'absence de migration)
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
Frédéric Péters a écrit :
Désolé si ça semble un revirement mais pour la lisibilité
Voilà, ça me va bien comme ça, j'avais tapé un truc similaire au timestamp principal du hobo.json sans réfléchir.
Du coup avec une mini gestion d'erreur pour ne pas planter sur les 'vieux' timestamp, juste ne rien afficher.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-environment-store-legacy-urls-timestamp-as-a-strftim.patch 0001-environment-store-legacy-urls-timestamp-as-a-strftim.patch ajouté
- Fichier 0002-environment-show-legacy-urls-63621.patch 0002-environment-show-legacy-urls-63621.patch ajouté
(avec les patchs c'est mieux)
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Go.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ad06b828b146696945623844c30ecbb67369f8b6 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Tue Apr 19 18:22:35 2022 +0200 environment: show legacy urls (#63621) commit a6e871d60c79a5e8ed63c0fe2f3f043a47c2927e Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Tue Apr 19 18:21:17 2022 +0200 environment: store legacy urls timestamp as a strftime output (#63621)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
environment: store legacy urls timestamp as a strftime output (#63621)