Produits Entr'ouvert » Passerelle

Pour infos, les URLs utilisées dans les formulaires :

[passerelle]/data/cg14-solis-communes/jsonp
[passerelle]/data/cg14-solis-lieux/jsonp?commune=[var_papier_code_commune]
[passerelle]/data/cg14-solis-lieux/jsonp?commune=[var_papier_excode_commune]
[passerelle]/data/cg14-solis-referentiel-caisse-de-retraite/jsonp
[passerelle]/data/cg14-solis-referentiel-etablissements/jsonp
[passerelle]/data/cg14-solis-referentiel-organismes-de-tutelle/jsonp

et dans les workflows :

[passerelle]/data/cg14-solis-communes/jsonp
[passerelle]/data/cg14-solis-dispatch/[form_var_code_commune_raw]/json?apikey=[apikey]
[passerelle]/data/cg14-solis-dispatch/[form_var_excode_commune_raw]/json?apikey=[apikey]
[passerelle]/data/cg14-solis-dispatch/[form_var_papier_code_commune_raw]/json?apikey=[apikey]
[passerelle]/data/cg14-solis-dispatch/[form_var_papier_excode_commune_raw]/json?apikey=[apikey]
[passerelle]/data/cg14-solis-homonymes/json?apikey=[apikey]&nom=[form_var_nombeneficiaire]&prenom=[form_var_prenombeneficiaire]&dn=[form_var_dnnbeneficiaire]
[passerelle]/data/cg14-solis-homonymes/json?apikey=[apikey]&nom=[form_var_nomconjoint]&prenom=[form_var_prenomconjoint]&dn=[form_var_dnnconjoint]
[passerelle]/data/cg14-solis-lieux/jsonp?commune=[var_code_commune]
[passerelle]/data/cg14-solis-referentiel-caisse-de-retraite/jsonp
[passerelle]/data/cg14-solis-referentiel-etablissements/jsonp
[passerelle]/data/cg14-solis-referentiel-organismes-de-tutelle/jsonp
[passerelle]/repost/solis-integration/json?apikey=[apikey]

Et dans la synchro :

repost/solis-suivi-visite/json?apikey=...
repost/solis-suivi-presentation-commission/json?apikey=...
repost/solis-suivi-decision-commission/json?apikey=...

Il faut pour chaque URL reproduire un comportement (input/output) identique, et indiquer la nouvelle URL à poser le cas échéant.

Par exemple POST /repost/solis-integration/json?apikey=[apikey] deviendra quelque chose comme POST /solis_apa/recette/demande/?orig=...&signature=...

Chouette, merci.

• remplacer _('Solis') par _('Solis APA') (solis est le nom d'une solution générique pour plein d'autres choses)
• dans utils.py les import du départ, on voit trop qu'ils viennent d'un copié collé ;)
• nettoyer le config.py pour ne laisser que les dicos CONCILIATION ; peut-être renommer ce fichier "conciliation.py" et y reprendre tout le code de utils.py, qui disparaitra
• mettre les commentaires en anglais ;)
• au moment de faire ce genre d'analyse du résultat :
  

          ret = response.json()
          l = []
          count = int(ret['ReferentialOutputWS']['Entries']['@count'])
          name = ret['ReferentialOutputWS']['Entries']['@name']
  

  
  il faut d'abord vérifier que ret contient bien les clés attendues, que c'est une réponse dans le bon format, et raiser une erreur "claire" en cas de pépin (au lieu d'un simple "KeyError" si on pose ce code). C'est la source de problème la plus importante actuellement en prod.

• factoriser le code qui gère les remontées caisse_retraite/organisme_tutelle/get_etablissement dans une seule vue referentiel/(?P<reference_name>[\w-+])/

Il faudrait pas retirer celui de passerelle/apps/ ?

Et l'important, c'est moins le déplacement de passerelle.apps vers passerelle.contrib que la réécriture intégrale, pour notamment ne plus dépendre de mako, non ?

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

Il faudrait pas retirer celui de passerelle/apps/ ?

On a nommé ce nouveau connecteur "solis_apa" car il s'agit d'un connecteur pour la gestion APA dans Solis (qui est un socle bien plus générique). Et on met dans contrib parce que ça reste assez lié au CG14 malgré tout (mais ça se voit pas... on pourrait donc mettre dans apps aussi).

Je préfère cependant ne pas supprimer l'ancien apps/solis tout de suite, tant que tout n'est pas recetté par cg14.

Et l'important, c'est moins le déplacement de passerelle.apps vers passerelle.contrib que la réécriture intégrale, pour notamment ne plus dépendre de mako, non ?

C'est le cas je crois ; j'ai seulement demandé à Josué de comparer les output de l'ancien connecteur (mako) avec le sien.

Tout mon commentaire, en fait, il est sur l'intitulé du commit, "migrate solis apa connector to contrib", qui me semble éloigné de ce qu'il contient.

• j'ai l'impression qu'on pourrait se passer du zip_code dans le modèle ? (ça serait plus "propre", je trouve). Mais le laisser au moindre doute, donc le laisser pour l'instant, allez...
• idem pour "application" : pourquoi en faire un paramètre ?
• je n'ai pas compris la property "headers", elle sert à quoi ? (ça peut pas être un simple dictionnaire quelque part ?)
• faire un peu plus attention à pep8 (y'a des espaces en trop ou qui manquent par ci par là, genre 'Accept':'application/json'), on n'est pas sur emacs, quand même :)

Pour le reste ça a l'air ok.

Thomas Noël a écrit :

relecture :

• j'ai l'impression qu'on pourrait se passer du zip_code dans le modèle ? (ça serait plus "propre", je trouve). Mais le laisser au moindre doute, donc le laisser pour l'instant, allez...
• idem pour "application" : pourquoi en faire un paramètre ?

Oui, j'ai éssayé d'etre un peu génerique

• je n'ai pas compris la property "headers", elle sert à quoi ? (ça peut pas être un simple dictionnaire quelque part ?)

yep pas faux.

• faire un peu plus attention à pep8 (y'a des espaces en trop ou qui manquent par ci par là, genre 'Accept':'application/json'), on n'est pas sur emacs, quand même :)

Pour le reste ça a l'air ok.

Allez hop, on y va ! Ack.