Labels de financement

[ggounot]

Implémentation de l´ADR sur les labels de financement #43.

Back-end

Modèles

• Création du modèle FundingLabel basé sur EnumModel ;
• Création d'une page admin pour le modèle FundingLabel ;
• Service : ajout d'un champ M2M FundingLabel ;
• SavedSearch : ajout d'un champ M2M FundingLabel.

Sérialiseurs

• Création du sérialiseur FundingLabelSerializer ;
• ServiceSerializer : ajout des champs funding_labels et funding_labels_display, sur le modèle des autres champs EnumModel ;
• SavedSearchSerializer : ajout des champs funding_labels et funding_labels_display, sur le modèle des autres champs EnumModel.

Logique de recherche

• _get_dora_results() : utilisation du filtre sur labels de financement ;
• _get_dora_results() : retour d'un dictionnaire de métadonnées concernant la recherche, contenant la liste des labels de financement associés aux résultats de recherche. Fait via une requête plutôt que par parcours des résultats par souci d'optimisation ;
• search_service() : retour le dictionnaire de métadonnées retourné par _get_dora_results(). À noter qu'aucun filtrage sur labels de financement ni récupération de métadonnées n'est fait pour la recherche DI.

Mapping des données DI

• map_search_result() : ajout du funding_labels ayant pour valeur fixe une liste vide ;
• map_service() : ajout des champs funding_labels et funding_labels_display ayant chacun pour valeur fixe une liste vide.

Vue

• search_services() : retour de la liste des labels de financement associés aux résultats.

Migrations

• Ajout des migrations de modèles de données ;
• Ajout d'une migration de création des labels de financement précédemment stockés dans un fichier JSON côté front-end.

Tests

Services (test_services.py)

• Adaptation de tests en prenant en compte de le champ funding_labels retourné par la recherche ;
• Ajout d'un test d'impossibilité de modifier les labels de financement d'un service ;
• Ajout de tests de recherche avec filtre sur labels de financement.

Recherches sauvegardées (test_services_saved_searchs.py)

• Ajout d'un test de vérification d'enregistrement des sous-catégories ;
• Ajout d'un test de vérification d'enregistrement des labels de financement.

Front-end

Types

• Ajout du type FundingLabel incluant le label et la valeur, à la différence des autres types reflétant EnumModel qui ne représentent que la valeur.

Page des résultats de recherche

• Utilisation des labels de financement donnés par le back-end en lieu et place de ceux donnés par le fichier JSON ;
• foundFundingLabels représente les labels de financement disponibles pour les résultats de recherche (affichés dans les filtres) tandis que fundingLabels représente les labels de financement sélectionnés pour filtrer les résultats.

Fiche service

• Affichage de Financé par : suivi de la liste des labels de financement associés séparés par des virgules.

Recherches sauvegardées

• Affichage des filtres de labels de financement s'ils existent ;
• Mise à jour de l'URL pour qu'elle inclus les filtres de labels de financement.

Autres

• Suppression de la constante FUNDED_SERVICES qui stockait toutes les labels de financements avec les slugs des services associés.

[ggounot]

Implémentation backend des labels financement

[jbuget]

Désolé, mais je n'ai rien repéré de croustillant ou critique. Moi, le débutant en Django, j'ai découvert des petits trucs sympas (le __in pour les filtres / recherche). Bravo pour le travail. J'ai surtout noté des "mercis" que tu peux resolve @ggounot .

A la rigueur, le seul point de difficulté de compréhension, c'est le nommage de la variable foundFundingLabels qui est dur à se remémrorer / comprendre. P-ê que ça aurait été plus simple pour moi avec un availaibleFundingLabels mais possible que j'ai raté un détail de compréhension.

La description complète de la PR et la lecture avant review m'a été hyper précieuse et utile.

🚀 LGTM !

[ggounot]

Désolé, mais je n'ai rien repéré de croustillant ou critique. Moi, le débutant en Django, j'ai découvert des petits trucs sympas (le __in pour les filtres / recherche). Bravo pour le travail. J'ai surtout noté des "mercis" que tu peux resolve @ggounot .

A la rigueur, le seul point de difficulté de compréhension, c'est le nommage de la variable foundFundingLabels qui est dur à se remémrorer / comprendre. P-ê que ça aurait été plus simple pour moi avec un availaibleFundingLabels mais possible que j'ai raté un détail de compréhension.

La description complète de la PR et la lecture avant review m'a été hyper précieuse et utile.

🚀 LGTM !

Merci beaucoup pour la review 🙏

Je n'étais moi-même pas satisfait du nom foundFundingLabels. Je l'ai renommée availaibleFundingLabels comme tu le proposes. C'est mieux ainsi. 2085d36

[jbuget]

Merci pour la prise en comptes des retours 🙏 LGTM