Projet

Général

Profil

Bug #23939

API: obtenir l'URL en FO et en BO du nouveau formdata pour un nouveau formdata

Ajouté par Benjamin Dauvergne il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Le cas d'usage où j'aurai besoin de cela est pour l'intégration de Maarch dans
Welco, car il est prévu de notifier Maarch de l'identifiant du nouveau formdata
ainsi que de son URL backoffice, pour éviter un roundtrip supplémentaire je
trouverai utile d'obtenir ces URLs dès la soumission.


Fichiers

Révisions associées

Révision 70c3a82d (diff)
Ajouté par Benjamin Dauvergne il y a presque 6 ans

api: return formdata URLs in ApiFormdefDirectory.submit() (#23939)

So that we do not need another roundtrip to have those informations.

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Je pouvais aussi renvoyer la même chose que sur un GET, j'ai choisi de ne pas
aller plus loin que mon besoin immédiat mais j'ai respecté le nom du champ donc
on peut très bien plus tard tout renvoyer (il me semble aussi
FormData.get_json_export_dict() ne renvoye pas de "url_backoffice", à mon avis
c'est un manque).

#2

Mis à jour par Frédéric Péters il y a presque 6 ans

Il faut mettre à jour la documentation (help/fr/api-fill.page) et inclure un test.

Aussi, même si les usages sont variés, je nommerais ça backoffice_url plutôt que url_backoffice. Et je retournerais aussi une api_url

#4

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Frédéric Péters a écrit :

Il faut mettre à jour la documentation (help/fr/api-fill.page) et inclure un test.

Aussi, même si les usages sont variés, je nommerais ça backoffice_url plutôt que url_backoffice. Et je retournerais aussi une api_url

C'est juste que partout ailleurs c'est dans ce sens (form_url_backoffice, site_url_backoffice).

#5

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Je ne suis pas certain de la construction des URLs d'API, si c'est ok j'ajoute les tests.

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

Tu as remis le même patch.

#8

Mis à jour par Frédéric Péters il y a presque 6 ans

Ok restent les tests à mettre en place.

#10

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

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

Mis à jour par Christophe Siraut il y a presque 6 ans

pour moi c'est ok.

#12

Mis à jour par Christophe Siraut il y a presque 6 ans

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

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Note pour moi même: en dehors des test connus pour ne pas passer dans tox, les tests passent. Donc poussable dès le dégel.

#14

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 70c3a82d9d2236519307820e76c523c0b3ec03f8 (HEAD -> master, origin/master, origin/HEAD)
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat May 19 16:05:39 2018 +0200

    api: return formdata URLs in ApiFormdefDirectory.submit() (#23939)

    So that we do not need another roundtrip to have those informations.

#15

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF