Development #17163
json data store : fournir creation_datetime / last_update_datetime dans les résultats.
0%
Description
-
Fichiers
Historique
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-jsondatastore-return-datetimes-in-results.patch 0001-jsondatastore-return-datetimes-in-results.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Valentin Deniaud
- Patch proposed changé de Non à Oui
Est-ce qu'il y a besoin d'un test ? Est-ce que le format par défaut passe ou il vaut mieux mettre un coup de strftime pour que ce soit plus joli ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
Oui pour le test.
Pour les datetimes, ils devraient être sérialisés automatiquement, pas besoin de str().
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-jsondatastore-return-datetimes-in-results.patch 0001-jsondatastore-return-datetimes-in-results.patch ajouté
J'ai enlevé les str et j'ai rajouté un test. Il a une bizarrerie parce que django rest framework ne formate pas les dates comme isoformat de datetime : il tronque les ms (==> ne pas en mettre) et il met des Z aux timezone UTC (==> enlever les Z...).
Mis à jour par Frédéric Péters il y a plus de 4 ans
django rest framework
La sérialisation devrait être celle de JSONEncoder dans passerelle/utils/jsonresponse.py, curieux d'avoir un Z qui apparait.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Mais plutôt que faire ce mock, tu peux juste comparer avec l'objet créé dans la base.
Mis à jour par Frédéric Péters il y a plus de 4 ans
La sérialisation devrait être celle de JSONEncoder dans passerelle/utils/jsonresponse.py, curieux d'avoir un Z qui apparait.
Sauf que là ça prend juste le time.struct_time, je n'ai rien dit.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Mon deuxième commentaire, comparer l'objet plutôt que mocker() un faux datetime, par contre, reste valable.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
En fait c'est Django tout court qui formate comme ça, pas DRF https://docs.djangoproject.com/en/2.2/topics/serialization/#djangojsonencoder. Mais je vais faire comme tu dis, c'est clairement mieux.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-jsondatastore-return-datetimes-in-results.patch 0001-jsondatastore-return-datetimes-in-results.patch ajouté
Voilà.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit eb1c4ad699d5ff5dd26bd58312366beee6cd1c0d Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 2 14:54:55 2019 +0200 jsondatastore: return datetimes in results
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
C'est en prod depuis longtemps.