feat: handle multiple extra fields filters #575
Conversation
|
@johanricher ici, j'ai pris un raccourci pas idéal. Explication : Imaginons, un jeu de donnée ayant 2 filtres complémentaires (fictifs) :
Si l'utilisateur sélectionne : Oui pour "Est-ce que vous voulez voir les jeux de données confidentiels " et Non pour "Est-ce que vous voulez voir les jeux de données sans restriction géographiques" L'on affichera les jeux de données qui soit réponde au critère "Oui" pour "Est-ce que vous voulez voir les jeux de données confidentiels" OU qui réponde au critère "Non" pour ""Est-ce que vous voulez voir les jeux de données sans restriction géographiques" Dans la logique, je pense qu'il faudrait faire en sorte d'afficher uniquement les jeux de données qui réponde aux 2 critères ("Oui" pour "Est-ce que vous voulez voir les jeux de données confidentiels" ET qui réponde au critère "Non" pour ""Est-ce que vous voulez voir les jeux de données sans restriction géographiques") Mon problème, c'est que techniquement, ce n'est pas quelque chose de trivial. J'ai mis à contribution Mathieu et Florimond pour m'aider à faire une requête SQL qui permettrait de répondre à ce besoin. Est-ce que garder pour le moment la logique du "OU" pourrait faire l'affaire ? (et que l'on crée une issue spécifique pour la logique du "ET") |
| @@ -28,3 +28,14 @@ export const last = <T>(arr: ArrayWithItems<T>): T => { | |||
|
|
|||
| export const removeEmptyValues = (items: Array<string | null>): Array<string> => | |||
| items.filter((item) => item) as string[]; | |||
|
|
|||
| export const chunk = <T = string>( | |||
There was a problem hiding this comment.
Inspiré de https://youmightnotneed.com/lodash
Volubyl
force-pushed
the
handle-multiple-extra-fields
branch
from
c23d7ca to
2ef8306
Compare
Je confirme qu'il faudrait que ce soit un ET, mais que pour l'instant le OU fait l'affaire. Tu crées un ticket qui détaille les améliorations à faire ? |
|
@julie-desvaux j'aurai besoin d'une reveiw par ici ! si tu veux on peu faire la relecture de la PR ensemble! :-) |
Suite de #563 , rel #290
Cette PR:
Apporte plusieurs améliorations :
1. Amélioration de l'affichage des filtres
l'affichage était pas terrible quand il y avait plusieurs filtres. Ceux-ci s'affichaient en une seule longue colonne.
Cette PR fait en sorte que les filtres soient affichés en plusieurs colonnes
2. Donne la possibilité de filtrer la recherche en fonction de plusieurs filtres de champs complémentaires
Maintenant, un utilisateur peut sélectionner plusieurs filtres de champ complémentaire et la recherche sera effectuée en fonction de chacun d'eux.
Pour tester
Aller sur la page de recherche et utiliser les filtres de recherches