Project

General

Profile

Development #37481

Afficher les schémas JSON déclarés pour les endpoints POST

Added by Benjamin Dauvergne about 1 month ago. Updated 28 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
06 Nov 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Le rendu est fait via un templatetag (les schéma étant récursif il n'est pas super pratique d'utiliser des templates pour cela).

Le rendu des paramètres "classiques" (query-string) est aussi modifié :
  • supression du paramètre "implicite" post_data, le schéma le remplace
  • création d'une section "Query parameters"
  • ajout du type (par défaut "string", float et integer sont possibles)

0001-misc-render-request-body-s-JSON-schema-35818.patch View (9.73 KB) Benjamin Dauvergne, 06 Nov 2019 10:20 AM

0001-misc-render-request-body-s-JSON-schema-37481.patch View (9.73 KB) Benjamin Dauvergne, 06 Nov 2019 11:19 AM

Screenshot-2019-11-12 Passerelle.png View (16.7 KB) Emmanuel Cazenave, 12 Nov 2019 03:42 PM

0001-misc-render-request-body-s-JSON-schema-37481.patch View (9.74 KB) Benjamin Dauvergne, 13 Nov 2019 09:37 AM

0002-atal-adapt-JSON-schema-for-display-37481.patch View (1.31 KB) Benjamin Dauvergne, 13 Nov 2019 09:37 AM

0001-misc-render-request-body-s-JSON-schema-37481.patch View (9.61 KB) Benjamin Dauvergne, 15 Nov 2019 12:24 PM

0002-to-fixup-add-translation.patch View (4.5 KB) Benjamin Dauvergne, 15 Nov 2019 03:05 PM

0001-misc-render-request-body-s-JSON-schema-37481.patch View (9.61 KB) Benjamin Dauvergne, 15 Nov 2019 03:05 PM

0003-to-fixup-missing.patch View (1014 Bytes) Benjamin Dauvergne, 15 Nov 2019 03:05 PM

0004-to-fixup-add-tests.patch View (2.42 KB) Benjamin Dauvergne, 15 Nov 2019 03:05 PM

38916

Related issues

Related to Passerelle - Development #37621: jsonschema : ne pas utiliser de références de schémas et utiliser draft4 Solution déployée 13 Nov 2019
Blocks Passerelle - Development #35818: mdel: démarche DDPACS Solution proposée 04 Sep 2019

Associated revisions

Revision 336b49b7 (diff)
Added by Benjamin Dauvergne about 1 month ago

misc: render request body's JSON schema (#37481)

History

#1 Updated by Benjamin Dauvergne about 1 month ago

#2 Updated by Benjamin Dauvergne about 1 month ago

#3 Updated by Benjamin Dauvergne about 1 month ago

Le numéro de commit n'est pas bon je corrigerai.

#5 Updated by Emmanuel Cazenave about 1 month ago

38916

Ça fait des trucs bizarre quand il y a un sous schéma (cf capture).

C'est pas que je tienne aux sous schémas, mais quand un endpoint doit accepter un fichier sérialisé par wcs, on est obligé de passer par un sous schéma (ou alors j'ai loupé un truc dans jsonschéma).

#6 Updated by Benjamin Dauvergne about 1 month ago

On peut simplement recopier/référencer (en Python) le schéma là où il est référencé (en JSON schema), ça n'aurait d'intérêt au niveau du code en tout cas que si on partageait ce sous schéma mais ça ne semble pas le cas.

#8 Updated by Emmanuel Cazenave about 1 month ago

  • Related to Development #37621: jsonschema : ne pas utiliser de références de schémas et utiliser draft4 added

#9 Updated by Emmanuel Cazenave about 1 month ago

En rebasant sur #37621, ça m'a l'air ok pour le rendu, tu dois pouvoir dégager ton deuxième patch et le workaround sur les histoires de 'required'.

#10 Updated by Benjamin Dauvergne about 1 month ago

Workaround pour required=True retiré ainsi que la modification au schéma Atal maintenant assurée par les commits de #37621.

#11 Updated by Thomas Noël about 1 month ago

Dans templatetags/passerelle.py :

#12 Updated by Benjamin Dauvergne about 1 month ago

Pour la traduction ça m'embête de traduire les types json (d'ailleurs j'ai changé bool en boolean pour coller au nom JSON schema).

#13 Updated by Thomas Noël about 1 month ago

  • Status changed from Solution proposée to Solution validée

Benjamin Dauvergne a écrit :

Pour la traduction ça m'embête de traduire les types json (d'ailleurs j'ai changé bool en boolean pour coller au nom JSON schema).

Allez peut-être juste traduire le "unknown" dans « format_html('<em>unknown {!r}</em>', original_schema) » mais juste si tu veux me faire plaisir.

Acke avec ou sans.

#14 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 336b49b7fde69cb1e15ae535b763e22b4e40197a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Nov 5 14:53:15 2019 +0100

    misc: render request body's JSON schema (#37481)

#15 Updated by Frédéric Péters 28 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF