https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-01-23T08:24:46ZRedmine Entr’ouvertPasserelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1523522019-01-23T08:24:46ZBenjamin Dauvergne
<ul><li><strong>Tracker</strong> changé de <i>Support</i> à <i>Development</i></li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1595332019-03-04T09:24:19ZSerghei Mihai
<ul><li><strong>Assigné à</strong> mis à <i>Serghei Mihai</i></li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1619842019-03-14T16:51:33ZSerghei Mihai
<ul><li><strong>Echéance</strong> mis à <i>21 mars 2019</i></li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629472019-03-20T11:05:48ZSerghei Mihai
<ul><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li><li><strong>Fichier</strong> <a href="/attachments/32772">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/32772/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li></ul><p>Je propose tous les documents qui ont un <code>type</code> défini.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629502019-03-20T11:09:56ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<ol>
<li>passerelle.apps.api_entreprise</li>
</ol>
</blockquote>
<p>Non, on ne fait pas ça.</p>
<blockquote>
<p>from urlparse import urljoin</p>
</blockquote>
<p>Sur les trucs évidents comme ceci, on essaie de penser à utiliser six.</p>
<blockquote>
<p>li.connector.apientreprise a::before {</p>
</blockquote>
<p>Non, la carte d'identité représente un usager, pas une asso ou entreprise.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629672019-03-20T12:35:42ZBenjamin Dauvergne
<ul></ul><p>Est-ce que pour les documents tu as pu voir si il y avait une régularité dans les types ? On pourrait en faire un référentiel à servir ? Si les nouveaux documents remplace les anciens ou si on se retrouve avec tout l'historique ?</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629802019-03-20T13:03:11ZSerghei Mihai
<ul></ul><p>Des tests que j'ai faits je n'ai vu que les types que tu as listés plus haut.<br />Les nouveaux ne remplacent pas les anciens, une nouvelle ligne est rajoutée avec le timestamp du nouveau document.</p>
<p>On pourrait imaginer un référentiel des types documents pour pouvoir les filtrer par date (en isoformat).</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629872019-03-20T13:13:56ZSerghei Mihai
<ul></ul><p>Déjà patch à jour avec les remarques de de Frédéric.</p>
<p>Pour l'icone j'ai choisi "fa-building-o" (<a class="external" href="https://fontawesome.com/cheatsheet#regular">https://fontawesome.com/cheatsheet#regular</a>) qui me paraît la plus proche d'une entreprise.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1629892019-03-20T13:15:26ZBenjamin Dauvergne
<ul></ul><p>Serghei Mihai a écrit :</p>
<blockquote>
<p>Des tests que j'ai faits je n'ai vu que les types que tu as listés plus haut.<br />Les nouveaux ne remplacent pas les anciens, une nouvelle ligne est rajoutée avec le timestamp du nouveau document.</p>
<p>On pourrait imaginer un référentiel des types documents pour pouvoir les filtrer par date (en isoformat).</p>
</blockquote>
<p>Idéalement dans une procédure on ne voudra que le dernier, mais à voir si en pratique on attache ça aux demandes ou on offre juste une fiche d'identité de l'association avec des liens vers tous les documents et un rendu de toutes les informations.</p>
Niveau asso on s'intéresse à trois choses:
<ul>
<li>les documents</li>
<li>les membres du bureau</li>
<li>d'éventuels coordonnées bancaires (je ne sais pas si ces APIs le porte)<br />Pour les entreprises je pense que seul le gérant intéresse les villes, mais peut-être le dernier bilan/CA.</li>
</ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1630782019-03-20T16:23:40ZSerghei Mihai
<ul></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
Niveau asso on s'intéresse à trois choses:
<ul>
<li>les membres du bureau</li>
</ul>
</blockquote>
<p>J'ai l'impression que c'est raté. L'API "associations RNA" (<a class="external" href="https://doc.entreprise.api.gouv.fr/?json#associations-rna">https://doc.entreprise.api.gouv.fr/?json#associations-rna</a>) ne fourni qu'un dirigéant, et sur l'exemple de la doc mais aussi celle de mon asso de vélo (waldec: W751205868) les champs sont à <code>null</code>.</p>
<blockquote>
<ul>
<li>d'éventuels coordonnées bancaires (je ne sais pas si ces APIs le porte)</li>
</ul>
</blockquote>
<p>Non.</p>
<blockquote>
<p>Pour les entreprises je pense que seul le gérant intéresse les villes, mais peut-être le dernier bilan/CA.</p>
</blockquote>
<p>Pour les entreprises on a les mandataires sociaux, rien sur le CA.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1630792019-03-20T16:28:43ZBenjamin Dauvergne
<ul></ul><p>Serghei Mihai a écrit :</p>
<blockquote>
<p>Benjamin Dauvergne a écrit :</p>
<blockquote>
Niveau asso on s'intéresse à trois choses:
<ul>
<li>les membres du bureau</li>
</ul>
</blockquote>
<p>J'ai l'impression que c'est raté. L'API "associations RNA" (<a class="external" href="https://doc.entreprise.api.gouv.fr/?json#associations-rna">https://doc.entreprise.api.gouv.fr/?json#associations-rna</a>) ne fourni qu'un dirigéant, et sur l'exemple de la doc mais aussi celle de mon asso de vélo (waldec: W751205868) les champs sont à <code>null</code>.</p>
<blockquote>
<ul>
<li>d'éventuels coordonnées bancaires (je ne sais pas si ces APIs le porte)</li>
</ul>
</blockquote>
<p>Non.</p>
<blockquote>
<p>Pour les entreprises je pense que seul le gérant intéresse les villes, mais peut-être le dernier bilan/CA.</p>
</blockquote>
<p>Pour les entreprises on a les mandataires sociaux, rien sur le CA.</p>
</blockquote>
<p>Yep je me doutais que les données étaient pourries, à mon avis ce serait plus intéressant d'avoir un bloc "association" dans toute démarche (et/ou un espace personne morale dans le portail agent) association où on affiche un maximum de choses, et le cas échéant la mention "manquante" pour certaines informations ; ça évitera les remontées des collectivités disant que ça ne marche pas ou que sais-je dans le cas où on choisirait plutôt de remplir des champs du formulaire avec ces informations.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1630832019-03-20T16:35:41ZSerghei Mihai
<ul></ul><p>Tu veux dire le résultat de l'appel à <a class="external" href="https://doc.entreprise.api.gouv.fr/?json#associations-rna">https://doc.entreprise.api.gouv.fr/?json#associations-rna</a> ?</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1634362019-03-22T14:13:16ZBenjamin Dauvergne
<ul></ul><p>Ces remarques globales étant faites, pour les dates ne faudrait-il pas normaliser pour des chaînes tous les timestamp (pour les identifier je dirai tous les clés qui commencent par "date_" et qui ont une valeur numérique, certaines finissent par <code>_timestamp</code> mais pas toute).</p>
<p>Pour les signatures je limiterai leur durée à quelque chose d'assez long, genre 1 semaine, on ne gène pas le travail mais on ne republie pas non plus tout leur référentiel.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1634462019-03-22T14:56:24ZSerghei Mihai
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Ces remarques globales étant faites, pour les dates ne faudrait-il pas normaliser pour des chaînes tous les timestamp (pour les identifier je dirai tous les clés qui commencent par "date_" et qui ont une valeur numérique, certaines finissent par <code>_timestamp</code> mais pas toute).</p>
</blockquote>
<p>Voici la liste des formats rencontrés:</p>
<pre>
"date_mise_a_jour": 1449183600,
"date_reference": "2014",
"date_creation_etablissement": 1108594800,
"date_fermeture": 1315173600,
"date_creation": 891381600,
"date_naissance": "1965-01-27",
"date_naissance": "",
"date_naissance_timestamp": 0,
"date_cessation": 1315173600,
"date_immatriculation": "1998-03-27",
"date_immatriculation_timestamp": 890953200,
"date_extrait": "21 AVRIL 2017",
"date": "2000-02-23",
"date_timestamp": 951260400,
"date_creation": "1993-02-11",
"date_declaration": "2013-06-28",
"date_publication": "1993-03-03",
"mise_a_jour": "2013-06-28"
</pre>
<p>Le isoformat et le timestamp sont le plus communs. Donc à essayer isoformat et timestamp. Et si ça matche pas on retourne la chaîne d'origine.</p>
<blockquote>
<p>Pour les signatures je limiterai leur durée à quelque chose d'assez long, genre 1 semaine, on ne gène pas le travail mais on ne republie pas non plus tout leur référentiel.</p>
</blockquote>
<p>Ok.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1634562019-03-22T15:24:17ZBenjamin Dauvergne
<ul></ul><p>Serghei Mihai a écrit :</p>
<blockquote>
<p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Ces remarques globales étant faites, pour les dates ne faudrait-il pas normaliser pour des chaînes tous les timestamp (pour les identifier je dirai tous les clés qui commencent par "date_" et qui ont une valeur numérique, certaines finissent par <code>_timestamp</code> mais pas toute).</p>
</blockquote>
<p>Voici la liste des formats rencontrés:</p>
<p>[...]</p>
</blockquote>
<p>Juste convertir les timestamp en dates, timezone France, le reste bizarre on y touche pas.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1634822019-03-22T20:18:01ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/32833">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/32833/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Hop.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1635802019-03-25T09:34:14ZBenjamin Dauvergne
<ul></ul><ul>
<li><code>context</code> et <code>object</code> ne devraient pas être des paramètres globaux du connecteur mais surchargeables à chaque appel (pour pouvoir le réutiliser dans différents contextes et pour différents objets, demande de sub, réservations de salle, etc..), idem contexte sur une plate-forme multicollectivité on pourrait considérer d'y passer le nom de la collectivité</li>
</ul>
<ul>
<li>Le code du endpoint document me parait suspect:
<ul>
<li>il reste un print</li>
<li>on parcourt une liste on vérifie une signature sur un paramètre et je ne vois aucune interaction entre les deux.</li>
</ul></li>
</ul>
<blockquote>
<p><a class="external" href="http://git.entrouvert.org/passerelle.git/tree/passerelle/apps/api_entreprise/models.py?h=wip/30010-api-entreprise&id=10b7df32c3c4b5c79baf078e54b493a2ffbf8fd7#n166">http://git.entrouvert.org/passerelle.git/tree/passerelle/apps/api_entreprise/models.py?h=wip/30010-api-entreprise&id=10b7df32c3c4b5c79baf078e54b493a2ffbf8fd7#n166</a></p>
</blockquote>
<ul>
<li>Dans le endpoint documents_associations pareil, on signe juste timestamp et type (je peux comprendre que ce soit les seuls références qu'on ait) mais ensuite on utilise que timestamp comme <code>id</code>, on a un datetime naïf créé qui est juste mis dans le json (je ne sais pas pourquoi ça passe, peut-être a-t-on un encodeur JSON spécifique comme dans w.c.s., mais je n'en dépendrai pas ici).
<ul>
<li>Pour simplifier j'encoderai directement l'URL dans le dico signé donc juste <code>signing.dumps({'url': document['url']})</code> il n'est pas nécessaire d'encoder tout le reste, ce endpoint est juste un proxy (on peut encoder object et context par contre, le contexte/objet d'une récupération de document dépendra ainsi du contexte/objet au moment de la récupération de la liste des documents)</li>
<li>Le endpoint document ne doit pas être protégé par <code>can_access</code> il est publique, ainsi les URLs qu'on aura pourront être directement mise dans des interfaces (fiche association)</li>
</ul></li>
</ul>
<ul>
<li>Ce serait peut-être bien si possible de donner un exemple des contenus renvoyés pour ne pas avoir à croiser la doc de API entreprise avec celle du connecteur pour les utilisateurs (si on peut donner un exemple de retour dans la description du endpoint ce serait bien)</li>
</ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1635822019-03-25T09:37:05ZBenjamin Dauvergne
<ul></ul><p>normalize_dates ignore les <code>timestamp</code> qu'on a justement sur le endpoint des documents (qui refait le job).</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1647962019-03-29T14:05:49ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/33018">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/33018/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>normalize_dates ignore les <code>timestamp</code> qu'on a justement sur le endpoint des documents (qui refait le job).</p>
</blockquote>
<p>J'ignorais le timestamp car je m'en servais comme <code>id</code>, mais ça peut être une date en effet.</p>
<p>Bonne idée pour utiliser <code>url</code>, <code>object</code> et <code>context</code> pour la récupération d'un fichier, merci.<br />Malheureusement dans <code>enpoint</code> on n'a rien pour donner des exemples de résultats des appels aux webservices.<br />Ça pourrait être objet d'un ticket à part.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1647972019-03-29T14:07:45ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-1 priority-4 priority-default" href="/issues/25432">Development #25432</a>: Étendre les possibilités de documentation des connecteurs</i> ajouté</li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1654852019-04-02T18:29:09ZBenjamin Dauvergne
<ul></ul><ul>
<li>un <code>print r</code> qui traîne</li>
<li>la validation d'url dans <code>document</code> me paraît inutile, on l'a signé c'est bon (et la signature on pourrait la valider hors de la boucle et s'arrêter là quand ça foire, enfin sans boucle le problème disparaît)</li>
<li>en relisant la doc d'API entreprise je vois que contexte, objet et recipent sont obligatoires et doivent être pertinents alors je ne les rendrais pas du tout globaux finalement (surtout pas recipient qui doit être le SIRET/WALDEC du bénéficiaire) et je les mettrais explicitement dans la signature des appels avec des indications sur les valeurs attendues données dans la doc, ça devrait rendre le code plus clair et virer get_context_and_object() qui est un peu obscur.
<ul>
<li>pour context : APS pour aide publique simplifié, MPS pour marché publique simplifié, etc..</li>
<li>pour object : le numéro de la demande dans w.c.s. ou d'un dossier</li>
<li>pour recipient: SIREN/SIRET/WALDEC du bénéficiaire</li>
</ul>
</li>
<li>j'ai toujours du mal avec le timestamp comme id, on a pas d'id faut faire avec, on utilisera jamais ce truc comme source donnée ça ne sert pas à grand chose, il faut juste renvoyer les données tels quels avec l'URL modifiée ; au mieux en imaginant un workflow où on aurait besoin d'un type de document particulier il faudrait un endpoint auquel on donne un type, un institution_id et qui renvoie le fichier le plus récent de ce type ou rien; parce que sinon j'ai du mal à imaginer un workflow où on viendrait utiliser les données renvoyées par document; on a un cahier des charges quelque part ? Je vois un ticket Arles mais qui ne précise pas les données qui les intéresse, si ça se trouve les documents ils s'en foutent.</li>
</ul>
<p>Ce qui est clair c'est qu'au niveau document ça ne peut servir qu'à l'agent pas à la personne qui remplit le formulaire, à moins d'avoir un endpoint fonctionnant comme je le dis (pour un type et un id, ça renvoie le document le plus récent au format JSON/fichier attaché ou rien), mais en fait je ne sais pas trop si ça marche le prefill sur les champs fichiers.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1665262019-04-08T17:32:48ZBenjamin Dauvergne
<ul></ul><p>Je serai pour ignorer la partie document et pousser tout de suite la partie qui a un intérêt immédiat pour Arles, donc juste ajouter les paramètres context, object, recipient et ce sera ok.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1667112019-04-09T22:51:03ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/33375">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/33375/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li></ul><p>Paramètres ajoutés.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1667792019-04-10T09:25:09ZBenjamin Dauvergne
<ul></ul><pre>
# try to parse isoformatted dates
try:
data[key] = datetime.fromtimestamp(int(data[key])).date()
except (ValueError, TypeError):
pass
</pre>
<p>Le commentaire est pas cohérent avec le code qui suit, on parse un timestamp ici pas du format iso, en fait la date iso est parfois fourni avec sa version timestamp... et tu convertis aussi les timestamp des documents que tu vas utiliser comme id, je ne les toucherai pas.</p>
Je ferai plutôt :
<ul>
<li>si champ commence par "date" et et ne finit pas par "timestamp" et contient un entier -> conversion vers "YYYY-mm-dd" </li>
<li>si champ "timestamp" création d'un nouveau champ "datetime" avec conversion vers "YYYY-mm-dd HH:MM:ss" localtime (et donc pour les documents je retournerai aussi ce datetime)</li>
<li>si timestamp 0, ne pas toucher (c'est une valeur particulière qui veut dire absent) ou alors convertir en None</li>
</ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1667802019-04-10T09:25:30ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1673702019-04-12T12:26:08ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/33442">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/33442/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>[...]</p>
<p>tu convertis aussi les timestamp des documents que tu vas utiliser comme id, je ne les toucherai pas.</p>
</blockquote>
<p>A ce sujet tu dis plus haut: "j'ai toujours du mal avec le timestamp comme id, on a pas d'id faut faire avec, on utilisera jamais ce truc comme source donnée ça ne sert pas à grand chose" :) mais allez, il nous faut bien un id.</p>
<blockquote>
Je ferai plutôt :
<ul>
<li>si champ commence par "date" et et ne finit pas par "timestamp" et contient un entier -> conversion vers "YYYY-mm-dd"</li>
</ul>
</blockquote>
<p>Ok.</p>
<blockquote>
<ul>
<li>si timestamp == 0, ne pas toucher (c'est une valeur particulière qui veut dire absent) ou alors convertir en None</li>
</ul>
</blockquote>
<p>Je préfère ne pas y toucher. C'est plus parlant que <code>None</code>.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1673912019-04-12T12:59:09ZBenjamin Dauvergne
<ul></ul><ul>
<li>Ça ne va pas marcher avec <code>"date_ref": "2014"</code> :<br /><pre>
if isinstance(data[key], int) or data[key].isdigit():
try:
data[key] = datetime.fromtimestamp(int(data[key])).date()
except (ValueError, TypeError):
pass
</pre><br />ça va faire un <code>.fromtimestamp(2014)</code> qui n'est pas du toute ce qu'on souhaite, j'en resterai à "c'est un entier" et c'est tout</li>
<li>Ça on aimerait conserver le préfixe de la clé je pense :<br /><pre>
if key.endswith('timestamp'):
# timestamps can be integers or strings
# convert only if positive values
if int(data[key]) > 0:
try:
object_datetime = make_aware(datetime.fromtimestamp(int(data[key])))
except (ValueError, TypeError):
pass
if object_datetime is not None:
data['datetime'] = object_datetime
</pre><br />et donc plutôt <br /><pre>
else:
data[key[:-len('timestamp)] + 'datetime'] = object_datetime
</pre><br />en virant la suite.</li>
</ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1673932019-04-12T13:03:42ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1675902019-04-15T07:55:53ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/33474">0001-api_entreprise-add-initial-connector-30010.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/33474/0001-api_entreprise-add-initial-connector-30010.patch">0001-api_entreprise-add-initial-connector-30010.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>J'ai du rêver qu'il y avait des timestamps sous forme de chaîne.<br />Bonne idée de garder de prefixer le <code>datetime</code> par le nom du champ.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1677022019-04-15T12:12:45ZBenjamin Dauvergne
<ul></ul><p>Une dernière chose, tu ne retournes pas le datetime, des documents, qu'on vient de créer; il faudrait :<br /><pre>
+ item['datetime'] = item['datetime']
</pre></p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1677122019-04-15T12:23:26ZSerghei Mihai
<ul></ul><p>Si. Dans les tests:</p>
<pre>
for document in data:
assert 'id' in document
assert 'text' in document
assert 'url' in document
assert 'type' in document
assert 'datetime' in document
</pre> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1677162019-04-15T12:27:08ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Au temps pour moi. Ack.</p> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1677202019-04-15T12:32:21ZSerghei Mihai
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit 16100aa2fb5aa18f01231f6ce361bcbc91ccb3b1 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Mar 19 15:37:04 2019 +0100
api_entreprise: add initial connector (#30010)
</pre> Passerelle - Development #30010: connecteur API entreprisehttps://dev.entrouvert.org/issues/30010?journal_id=1694492019-04-23T09:48:27ZSerghei Mihai
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>