Projet

Général

Profil

Development #67210

rsa13: ajouter les endpoints pour la phase 1.3

Ajouté par Benjamin Dauvergne il y a presque 2 ans. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
10 juillet 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

  • [RSA] Mise à disposition des accompagnateurs une liste des brsa sortis d'une leur plateforme depuis moins de 90 jours : #66285
  • [RSA] Connecteur sous-actions : #66139

Fichiers


Demandes liées

Lié à Passerelle - Development #68768: rsa13 : ajouter des paramètres à la description du endpoint sous-actionFermé06 septembre 2022

Actions
Lié à Passerelle - Development #68769: rsa13 : corriger l'URL d'exemple du endpoint .../beneficiaire/sorti/csv/Fermé06 septembre 2022

Actions

Révisions associées

Révision 10103f9d (diff)
Ajouté par Benjamin Dauvergne il y a plus d'un an

rsa13: add beneficiaire/sorti/csv endpoint (#67210)

Révision 99d70b3a (diff)
Ajouté par Benjamin Dauvergne il y a plus d'un an

rsa13: add sous-action endpoint (#67210)

Révision 60dc8f36 (diff)
Ajouté par Benjamin Dauvergne il y a plus d'un an

rsa13: fix wrong id schema in nomenclatures endpoint (#67210)

Historique

#5

Mis à jour par Serghei Mihai il y a presque 2 ans

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

Mis à jour par A. Berriot il y a presque 2 ans

C'est plus une question pour ma culture personnelle : dans le premier patch, les lignes :

    names = [col if isinstance(col, str) else col[0] for col in csv_columns]
    titles = [col if isinstance(col, str) else col[1] for col in csv_columns]

C'est parce que RSA13_BENEFICIAIRE_SORTI_CSV_COLUMNS peut-être une liste de tuples ? Dans quelle situation ça peut se produire ?

Dans le second patch, pour la définition du endpoint, je vois

                        'id': {

                            'enum': [0, 1],

                        },

Et dans la spec, ID a l'air de ressembler a "id": "AM1". C'est normal ?

#7

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

Agate Berriot a écrit :

C'est plus une question pour ma culture personnelle : dans le premier patch, les lignes :
[...]

C'est parce que RSA13_BENEFICIAIRE_SORTI_CSV_COLUMNS peut-être une liste de tuples ? Dans quelle situation ça peut se produire ?

Pour un précédent CSV on m'a demandé de pouvoir changer le nom des colonnes sans changer leur web-service alors je fais ça partout maintenant, pour éviter d'avoir à modifier du code si la demande revient.

Dans le second patch, pour la définition du endpoint, je vois

[...]

Et dans la spec, ID a l'air de ressembler a "id": "AM1". C'est normal ?

Ah oui c'est complètement con, je ne sais pas d'où ça sort un copier/coller malheureux, merci.

#8

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

L'erreur était effectivement déjà là dans le endpoint nomenclature, je corrige ça dans le dernier commit (c'est assez invisible car on ne valide pas nos réponses par rapport à nos propres schémas, on devrait éventuellement le faire au moins dans les tests, ça nous donnerait l'assurance que tout ça reste cohérent et documenté).

#9

Mis à jour par A. Berriot il y a presque 2 ans

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

Mis à jour par A. Berriot il y a presque 2 ans

Benjamin Dauvergne a écrit :

Agate Berriot a écrit :

C'est plus une question pour ma culture personnelle : dans le premier patch, les lignes :
[...]

C'est parce que RSA13_BENEFICIAIRE_SORTI_CSV_COLUMNS peut-être une liste de tuples ? Dans quelle situation ça peut se produire ?

Pour un précédent CSV on m'a demandé de pouvoir changer le nom des colonnes sans changer leur web-service alors je fais ça partout maintenant, pour éviter d'avoir à modifier du code si la demande revient.

D'accord, merci pour la clarification :)

#11

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 60dc8f362d01aecb034fee15bee432d8471445f8
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Jul 27 11:30:38 2022 +0200

    rsa13: fix wrong id schema in nomenclatures endpoint (#67210)

commit 99d70b3a02bce18d414e3e29439b0863f45fb395
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sun Jul 10 11:48:43 2022 +0200

    rsa13: add sous-action endpoint (#67210)

commit 10103f9d45110f0b7c31d9c8f33e93fad26afd9c
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sun Jul 10 11:43:37 2022 +0200

    rsa13: add beneficiaire/sorti/csv endpoint (#67210)
#12

Mis à jour par Transition automatique il y a plus d'un an

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

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a plus d'un an

  • Lié à Development #68768: rsa13 : ajouter des paramètres à la description du endpoint sous-action ajouté
#14

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a plus d'un an

  • Lié à Development #68769: rsa13 : corriger l'URL d'exemple du endpoint .../beneficiaire/sorti/csv/ ajouté
#15

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF