Projet

Général

Profil

Development #17163

json data store : fournir creation_datetime / last_update_datetime dans les résultats.

Ajouté par Frédéric Péters il y a presque 7 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
23 juin 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

-


Fichiers

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 4 ans

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 ?

#2

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().

#3

Mis à jour par Valentin Deniaud il y a plus de 4 ans

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...).

#4

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.

#5

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.

#6

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.

#7

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.

#8

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.

#10

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#11

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

#12

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.

Formats disponibles : Atom PDF