Produits Entr'ouvert » Passerelle

Ack. Juste quelques suggestions à prendre ou a laisser.

• lignes non utilisées :
  

  HTTP_MOCKS = {
      'type-emploi': {
          'path': r'^/.*/data/type_emploi$',
          #'query': 'viewIntegrationName=api_publik',
  

• fixture mock_500 non définie
• fixture http_mock surchargée
  

  def test_data_annonce_error_500(self, resource, http_mock, caplog):
  def test_pdf_error_500(self, resource, http_mock, caplog):
  

• import non utilisé :
  

  from django.core.exceptions import ValidationError
  

• log_requests_errors = False
  Je vois qu'il est aussi positionné sur smart et que ça n'a pas posé de problème pour déboguer mais je me demande quand même à quoi ça sert ici.

• def announce(
  J'aurais mis un 's', mais j'avoue que je m'y perd : update_referentiels met à jour les annonces et les référentiels, et announce renvoie les annonces et les référentiels.
  Sinon, peut-être les renommer update et search ?

• À la lecture de la description du endpoint datasource via l'IHM on imagine qu'il s'utilise avec "ds?name=XXX" alors qu'en fait il faut utiliser le format "ds/XXX/".
  Idem pour les PDF : "announce/123/pdf/".

• Pour la recherche textuelle, rien ne précise dans le ticket client sur quel champ la faire, et je me demande s'il ne faudrait pas l'étendre à la description de l'annonce.

• schémas JSON des réponse,
• long_description pour les 3 endpoints (précisant les référentiels dispo et leur nom pour data_sources)
• affichage de la date de renière mise à jour pour les annonces et les sources de donnée sur la page du connecteur

• ajout de la description du champ data_sources au schéma de la réponse du endpoint announce (ça permet d'afficher la page de recherche en un appel)
• ajout du filtrage par collectivite (les annonces seront vraisemblablement affichées pour chaque collectivité sur des pages différentes)

C'est chouette l'ajout de patternProperties,
https://json-schema.org/understanding-json-schema/reference/object.html#pattern-properties
mais je l'aurais bien vu dans un commit à part parce que c'est pas évident de comprendre comment on doit l'utiliser.

Là j'ai l'impression qu'on en fait un usage détourné en ne renseignant pas de regex, et du coup j'ai un affichage approximatif (pattern //):

data_sources : object
    pattern // : array object
        id : string
        text : string

Allez tout bien séparé.

Changement au rendu des patternProperties.

Super, merci.

commit 91e7c9afcff26e1ac88a2c70d936e2a40ef35815
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Apr 19 15:20:00 2022 +0200

    add toulouse_foederis connector (#64313)

commit 4074f8222d702d28b1a6d891fcfd71da60776a4d
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Apr 21 16:35:30 2022 +0200

    templatetags: add support for patternProperties JSON schema (#64313)

commit dea4352c91a49633c4c8c199c5b04b7b564980ba
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Apr 21 16:37:53 2022 +0200

    utils/json: add datasource_schema helper (#64313)

commit 132185c03db91133f61349efcfb53d70cc720143
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Apr 21 12:10:37 2022 +0200

    utils/json: move response_schema from rsa13 (#64313)

commit 2f729a7fb64939b57cdffe913e8a044ca03c0f36
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Apr 19 15:25:16 2022 +0200

    misc: force message to be in string in _log (#64313)

    It allows translated strings in log calls.