feat(ui): ajout du filtre UAI (connu ou inconnu) #3278
Conversation
|
Bonus : voir pour trouver un moyen propre de ne pas dupliquer de code en créant un filtreYesNo paramétrable avec des libellés custom. |
| if (organismesFilters.etatUAI?.[0] === true) return item.uai; | ||
| if (organismesFilters.etatUAI?.[0] === false) return !item.uai; | ||
| }); | ||
| } |
There was a problem hiding this comment.
A priori, on ne gère qu'un seul filtre. Il faudrait que la sélection de tous les filtres soit équivalente à ne rien sélectionner pour rester cohérent avec les autres filtres.
There was a problem hiding this comment.
Du coup, ça veut dire que chaque organisme doit correspondre à au moins un filtre sur l'UAI.
En regardant de plus près, je me suis rendu compte que le filtre sur transmission avait à peu près la même tronche, mais fonctionne à cause du organismesFilters.transmission?.length === 1, car théoriquement dès que >= 2 params, c'est true + false donc tout donc = aucun filtre.
Mais sinon, y'a moyen de faire qqch qqch comme ça pour avoir un filtre complètement fonctionnel :D (j'ai modifié transmission, mais c'est pareil pour l'UAI) :
if (organismesFilters.transmission?.length && organismesFilters.transmission?.length > 0)
filteredOrganismes = filteredOrganismes?.filter((organisme) =>
organismesFilters.transmission?.some(filter => (!!filter && !!organisme.last_transmission_date) || (!filter && !organisme.last_transmission_date))
);There was a problem hiding this comment.
A priori, on ne gère qu'un seul filtre. Il faudrait que la sélection de tous les filtres soit équivalente à ne rien sélectionner pour rester cohérent avec les autres filtres.
C'est le cas normalement et c'est géré il me semble avec cette fonction 🤔
There was a problem hiding this comment.
Du coup, ça veut dire que chaque organisme doit correspondre à au moins un filtre sur l'UAI. En regardant de plus près, je me suis rendu compte que le filtre sur transmission avait à peu près la même tronche, mais fonctionne à cause du
organismesFilters.transmission?.length === 1, car théoriquement dès que >= 2 params, c'est true + false donc tout donc = aucun filtre.Mais sinon, y'a moyen de faire qqch qqch comme ça pour avoir un filtre complètement fonctionnel :D (j'ai modifié transmission, mais c'est pareil pour l'UAI) :
if (organismesFilters.transmission?.length && organismesFilters.transmission?.length > 0) filteredOrganismes = filteredOrganismes?.filter((organisme) => organismesFilters.transmission?.some(filter => (!!filter && !!organisme.last_transmission_date) || (!filter && !organisme.last_transmission_date)) );
Y'a effectivement une erreur sur la condition organismesFilters.etatUAI?.length qui doit etre === 1.
Après sur le code il y a effectivement une autre façon de l'écrire, ca change la lisibilité et évite le test sur organismesFilters.transmission?.length === 1
sbenfares
force-pushed
the
feature/ajout-filtre-uai
branch
from
46d7205 to
f27fe10
Compare
🚀 Prévisualisation |
|
🎉 This PR is included in version 3.61.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Vu avec @cathbaiona : ajout d'un filtre sur l'UAI connue ou inconnue sur la liste des organismes.
En attente décision produit @cathbaiona @nadinelouchart sur le fait que les admin seulement peuvent voir le filtre.