Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add filter orga in projects & projects in orgas #218

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fluidlog
Copy link
Contributor

Suite à la demande du Low-tech Lab, j'ai ajouté un filtre "Organisation" dans la liste des projets

image

Vu que la liste des organisations peut être longue, je l'ai mis en bas...

Puis, dans l'autre sens, j'ai ajouté un filtre "Projets" dans la liste des organisations.

image

J'ai testé, et ça filtre bien dans les deux cas.

@fluidlog fluidlog requested a review from mguihal December 29, 2024 19:33
@fluidlog fluidlog added low difficulty Easy issue (difficulty) ux Interface utilisateur medium priority frontend labels Dec 29, 2024
@simonLouvet
Copy link
Contributor

J'ai un doute. Si la vue d'une orga incorporait plus proprement ses projets, il n'y aurait pas besoin de filtrer les projets par orga. Filtrer les organisations par projet me semble également moins pertinent que naviguer sur le projet et de voir les orgas impliquées

@@ -30,6 +30,15 @@ const OrganizationFilterSidebar = () => {
filter={{}}
sort={{ field: 'pair:label', order: 'DESC' }}
/>
<ReferenceFilterTree
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attention, un ReferenceFilterTree n'est applcable que pour les thèmes, il faut utiliser ReferenceFilter pour les autres ressources non arborescentes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah oui, pardon, je vais en effet corriger ça...

Copy link
Collaborator

@mguihal mguihal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personnellement j'ai un gros doute sur la pertinence de rajouter un filter Projet sur la vue Organisation. As-tu un cas concret en tête ? Le but n'est pas de rajouter le plus de filtres possibles, mais de rajouter les plus pertinents et génériques à toutes les situations.

Vu que la liste des organisations peut être longue, je l'ai mis en bas...

Ca ca m'interpelle aussi. On va se retrouver avec une liste de plusieurs centaines d'éléments (qu'il faut charger) sur la page des projets, alors que 99% de ces orgas ne vont peut-être n'avoir aucun projet et donc être inutile. Ca va brouiller totalement l'interface pour les utilisateurs.

Dans un premier temps, si c'est une demande de LTL spécifique, alors il faut surcharger ces fichiers sur l'instance.

Ce qu'on pourrait faire de plus pérenne, c'est de pouvoir configurer par ressource les filtres à afficher dans la sidebar dans le fichier de configuration.

@fluidlog
Copy link
Contributor Author

fluidlog commented Jan 2, 2025

Je suis tout à fait d'accord pour inclure cela dans le fichier de configuration.

Pour vous donner un exemple, pour le LTL, je comprends que ce soit intéressant pour eux, c'est une demande de Guénolé, et je pense qu'il souhaite avoir une vue sur la liste de tous les projets, et de filtrer plutôt que d'aller sur la vue orga à chaque fois.

image

S'il n'y avait pas plusieurs orgas possibles, on aurait pu les indiquer dans le titre du projet (dans la liste)...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend low difficulty Easy issue (difficulty) medium priority ux Interface utilisateur
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants