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

[BO - Liste signalement] [Front] Gestion des filtres #2572

Merged
merged 13 commits into from
May 30, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented May 17, 2024

Ticket

#2121
#2590
#1605

Description

Spécification et maquettes
https://github.com/MTES-MCT/histologe/wiki/liste-signalements
https://xd.adobe.com/view/8ebe18a2-c737-48b6-af34-e79b9f189d9d-dbed/grid

Changements apportés

  • Ajout de nouveaux composants pour gérer l'autocompletion, champ recherche et les nombre pour la criticité
    * HistoAutoComplete.vue
    * HistoNumber.vue
    * HistoSearch.vue
  • Affichage et gestion des filtres selon le rôle
  • Ajout de donnée dans le web-service de widget-settings
  • Appels du WS avec filtres

Pré-requis

make npm-build

Pour ajouter des EPCI

make console app=load-epci

Mettre à jour la data de la la nouvelle colonne last_suivi_is_public qui accompagne last_suivi_at et last_suivi_by

make console app=update-signalement-lastsuivi

Supprimer le cache

make clear-pool pool="--all"

Tests

  • Tester tous les filtres en tant qu'admin
  • Tester tous les filtres en tant que RT (Responsable de Territoire)
  • Tester tous les filtres en tant qu'agent
  • Vérifier l'affichage des étiquettes de filtres actifs à côté du bouton "Masquer les options"
  • Vérifier que les étiquettes des filtres actifs restent visibles après le clic sur "Masquer les options"
  • Tester la suppression des étiquettes de filtres actifs
  • Vérifier que la liste des signalements se réinitialise ainsi que les champs de recherche lors du clic sur le bouton de réinitialisation

@sfinx13 sfinx13 force-pushed the feature/2121-handle-filter branch from 782fbee to cd73c12 Compare May 17, 2024 13:21
@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented May 21, 2024

Retours de test en SA :

  • est-ce que c'est normal d'avoir le bouton "afficher mes affectations uniquement" en SA ?
  • dans le champ Commune ou code postal, si je tape le nom d'une ville pour laquelle il n'y a pas de signalement, j'ai l'impression que ça ne marche pas
  • idem pour le champ EPCI
  • j'ai l'impression qu'il y a un souci sur le filtre "type de dernier suivi" je choisis "suivi partenaire", et un des 5 signalements a comme dernier suivi un suivi usager (http://localhost:8080/bo/signalements/00000000-0000-0000-2022-000000000004) et de lpus sur la liste ils ont tous le badge "suivi interne", mais quand j'ouvre les signalements tous les derniers signalements sont "visibles par l'usager" (alors je ne sais pas si le pb est côté liste ou côté fiche signalement) Et si je sélectionne "suivi usager" le signalement 2022-4 ne remonte pas.
  • La liste de partenaires est vide si je ne sélectionne pas un territoire
  • si je sélectionne le 13, j'ai seulement les partenaires 13-01 et 13-02 dans la liste (alors même que j'ai des affecctations avec d'autres partenaires du 13) mais ensuite la sélection marche bien
  • Pareil pour les étiquettes, je n'en vois que 2 alors qu'il y en a 3
  • Enfants - 6 ans, si je sélectionne "non renseigné, j'ai 3 signalements et quand je les ouvre j'ai "oui" pour enfants de - 6 ans : 00000000-0000-0000-2024-000000000002, 00000000-0000-0000-2023-000000000027 et 00000000-0000-0000-2024-000000000003

Tout le reste m'a semblé OK pour les SA (je ferai les tests avec les autres utilisateurs après)

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

J'ai pas mal de petit bug ou questions, voici la liste testé uniquement en super admin pour l'instant

  • Est-ce normal que les filtres territoire et statut ne soit pas à selection multiple ?

  • Bug sur les résultat de recherche par référence (ex 2024-650 sur copie de la prod) mais ca ne le fait pas toujours
    reference-ko

  • La recherche par code postal fonctionne uniquement si un département est sélectionné

  • EPCI :
    -- Si on en selectionne plusieurs a la suite le libellé du filtre se cumule
    epci
    -- pourquoi garder la valeur numérique ?
    -- Je ne comprend pas trop les résultat proposé, si je cherche "Alès" sans territoire sélectionné je n'ai pas de proposition, ca fonctione si je sélectionné le Gard avant.

  • Dommage que les acent ne soit pas ignoré pour la recherche autocomplete, si dans le territoire Gard je cherche "Ales" dans le champ commune ou EPCI je n'ai pas de résultat alors que "Alès" fonctionne

  • Les valeur champs date ne sont pas vidé visuellement quand on clique sur réinitialiser

  • Les filtre Min/Max (criticité) se sont pas comme sur la charte, et sous ce format je n'ai clairement pas compris leur critere sans aller lire les specs

  • Sur ces filtres Min/Max si je renseigne les deux puis que je les efface la sélection ne semble pas se mettre à jour

  • Peut etre hors sujet mais j'ai un crash quand je tente d'exporter les signalement

assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-list/store.ts Outdated Show resolved Hide resolved
src/Entity/Behaviour/TimestampableTrait.php Show resolved Hide resolved
src/Entity/Enum/SignalementStatus.php Show resolved Hide resolved
@sfinx13
Copy link
Collaborator Author

sfinx13 commented May 21, 2024

@hmeneuvrier

  • est-ce que c'est normal d'avoir le bouton "afficher mes affectations uniquement" en SA ?
  • Oups!! j'ai oublié de le gérer
  • dans le champ Commune ou code postal, si je tape le nom d'une ville pour laquelle il n'y a pas de signalement, j'ai l'impression que ça ne marche pas
  • La liste de partenaires est vide si je ne sélectionne pas un territoire
  • idem pour le champ EPCI
  • J'ai mal interprété la spec Pour le SA, les options se mettent à jour lorsqu'on sélectionne un territoire
    je vais tous charger
  • j'ai l'impression qu'il y a un souci sur le filtre "type de dernier suivi" je choisis "suivi partenaire", et un des 5 signalements a comme dernier suivi un suivi usager (http://localhost:8080/bo/signalements/00000000-0000-0000-2022-000000000004) et de lpus sur la liste ils ont tous le badge "suivi interne", mais quand j'ouvre les signalements tous les derniers signalements sont "visibles par l'usager" (alors je ne sais pas si le pb est côté liste ou côté fiche signalement) Et si je sélectionne "suivi usager" le signalement 2022-4 ne remonte pas.
  • Bien vu je check ça
  • si je sélectionne le 13, j'ai seulement les partenaires 13-01 et 13-02 dans la liste (alors même que j'ai des affecctations avec d'autres partenaires du 13) mais ensuite la sélection marche bien

j'ai pas réussi à reproduire, problème de chargement (erreur console ?)

  • Pareil pour les étiquettes, je n'en vois que 2 alors qu'il y en a 3

j'ai pas réussi à reproduire, problème de chargement (erreur console ?)

  • Enfants - 6 ans, si je sélectionne "non renseigné, j'ai 3 signalements et quand je les ouvre j'ai "oui" pour enfants de - 6 ans : 00000000-0000-0000-2024-000000000002, 00000000-0000-0000-2023-000000000027 et 00000000-0000-0000-2024-000000000003
  • Bien vu la requête se fait que sur la colonne nb_enfants_m6 => ancien signalement (a faire sur le champs json)

@sfinx13
Copy link
Collaborator Author

sfinx13 commented May 21, 2024

@numew

J'ai pas mal de petit bug ou questions, voici la liste testé uniquement en super admin pour l'instant

  • Est-ce normal que les filtres territoire et statut ne soit pas à selection multiple ?

J'ai refait une passe sur les spec territoire et status ne sont pas en sélection multiple

Le filtre multiple sur les territoires je ne vois plus trop l'utilité pour les SA (nous) avec metabase

Pour les statuts la question peut être reposé.

  • Bug sur les résultat de recherche par référence (ex 2024-650 sur copie de la prod) mais ca ne le fait pas toujours
    reference-ko
  • Ah oui comme on fait plusieurs requête à partir de 3 caractères c'est la derniere qui répond qui gagne, j'ai trouvé une solution assez simple grâce à Axios qui annule les requêtes précédentes lors d'une nouvelle requête
    Une recherche sur 202 mettra plus de temps à répondre que 2022-403

https://axios-http.com/fr/docs/cancellation

  • La recherche par code postal fonctionne uniquement si un département est sélectionné
  • EPCI :
    -- Si on en selectionne plusieurs a la suite le libellé du filtre se cumule
  • Mauvaise interprétation de la spec

epci

-- pourquoi garder la valeur numérique ?

  • Parce que je suis fainéant :-D

-- Je ne comprend pas trop les résultat proposé, si je cherche "Alès" sans territoire sélectionné je n'ai pas de proposition, ca fonctione si je sélectionné le Gard avant.

  • Dommage que les acent ne soit pas ignoré pour la recherche autocomplete, si dans le territoire Gard je cherche "Ales" dans le champ commune ou EPCI je n'ai pas de résultat alors que "Alès" fonctionne
  • Je travaillerai avec une base de prod et ajouterai des fixtures dans ce sens
  • Les valeur champs date ne sont pas vidé visuellement quand on clique sur réinitialiser
  • Effectivement
  • Les filtre Min/Max (criticité) se sont pas comme sur la charte, et sous ce format je n'ai clairement pas compris leur critere sans aller lire les specs

J'avoue que j'ai vu ce composant assez tard, faudrait un ticket à part pour faire ce composant car j'ai fait un truc minimaliste pour valider rapidement le fonctionnement du filtre

  • Sur ces filtres Min/Max si je renseigne les deux puis que je les efface la sélection ne semble pas se mettre à jour
  • Peut etre hors sujet mais j'ai un crash quand je tente d'exporter les signalement

Tu peux m'envoyer l'url stp merci

@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented May 23, 2024

  • si je sélectionne le 13, j'ai seulement les partenaires 13-01 et 13-02 dans la liste (alors même que j'ai des affecctations avec d'autres partenaires du 13) mais ensuite la sélection marche bien

pas reproduit avec les données de prod (bon il y a BEAUCOUP de partenaires et d'étiquettes et j'ai bien une scrollbar mais je n'ai pas compté)

EDIT : reproduit sur les fixtures en tant que RT du 01, il y a minimum 4 partenaires en base, mais je n'en vois que 2 :
image
C'est un souci d'affichage, car dans la réponse on a 4 partenaires :
image

Et pareil sur les étiquettes, on en a 3 dans la réponse, mais on n'en voit que 2 :
image

ça a l'air d'être un souci d'affichage sur petit nombre

@hmeneuvrier
Copy link
Collaborator

  • Les valeur champs date ne sont pas vidé visuellement quand on clique sur réinitialiser

toujours le cas

@emilschn
Copy link
Collaborator

emilschn commented May 23, 2024

Mes retours

1/ Il manque un label pour la criticité min / max ? c'est peut-être que ce que tu as dit ce matin à propos de ces champs number. Je me souviens que tu en as parlé, mais je ne me souviens plus du souci exact.

2/ Je comprends pas le champ de recherche avec "EPCI" en placeholder
De manière générale, c'est pas trop conseillé de mettre des champs sans label. C'est prévu ensuite ?

3/ Je ne trouve pas les labels de filtres sélectionnés très clair.
Par exemple : je sélectionne 2 filtres étiquettes (dont Urgent). Et ça marche Urgent +1
Je vois ce que c'est, mais c'est pas hyper clair.
Est-ce qu'on peut envisager d'avoir Etiquettes : Urgent, Péril ? (un peu comme la criticité)

4/ Les filtres n'ont pas encore été reportés sur l'export csv ? Ok pour ça, mais par contre, je ne vois pas de ticket pour le prendre en compte ensuite.

@sfinx13
Copy link
Collaborator Author

sfinx13 commented May 24, 2024

@emilschn

1/ Il manque un label pour la criticité min / max ? c'est peut-être que ce que tu as dit ce matin à propos de ces champs number. Je me souviens que tu en as parlé, mais je ne me souviens plus du souci exact.

Je proposait la création d'un nouveau composant conforme à la maquette dans un nouveau ticket
image

  • Ticket crée

2/ Je comprends pas le champ de recherche avec "EPCI" en placeholder De manière générale, c'est pas trop conseillé de mettre des champs sans label. C'est prévu ensuite ?

  • En plus des titres d'attributs, j'ai ajouté des labels sur les etiquettes
    image

A discuter en point dev/ux

  • Je peut mettre la mention "Rechercher par ...." avec un attribut title

3/ Je ne trouve pas les labels de filtres sélectionnés très clair. Par exemple : je sélectionne 2 filtres étiquettes (dont Urgent). Et ça marche Urgent +1 Je vois ce que c'est, mais c'est pas hyper clair. Est-ce qu'on peut envisager d'avoir Etiquettes : Urgent, Péril ? (un peu comme la criticité)

Pareil c'est dans la maquette, c'est le fait que les valeurs soient accessible en déroulant la liste

4/ Les filtres n'ont pas encore été reportés sur l'export csv ? Ok pour ça, mais par contre, je ne vois pas de ticket pour le prendre en compte ensuite.

  • Reporté

@hmeneuvrier
Copy link
Collaborator

  • Les valeur champs date ne sont pas vidé visuellement quand on clique sur réinitialiser

OK

@hmeneuvrier
Copy link
Collaborator

ça a l'air d'être un souci d'affichage sur petit nombre

je confirme, suivant la taille de l'écran, j'ai entre 2 et 7 items qui apparaissent

@sfinx13
Copy link
Collaborator Author

sfinx13 commented May 24, 2024

ça a l'air d'être un souci d'affichage sur petit nombre

je confirme, suivant la taille de l'écran, j'ai entre 2 et 7 items qui apparaissent

J'ai pas ces soucis d'affichage j'ai aussi testé sur firefox et je me suis mis aussi à plus de 100%
image

image

Copy link

sonarcloud bot commented May 27, 2024

Quality Gate Passed Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

C'est tout bon pour moi !

@emilschn
Copy link
Collaborator

Deux retours de test (il me manque peut-être quelque chose)
J'ai pourtant joué les migrations et les commandes recommandées en pré-requis.

1/ Au démarrage, j'ai un souci lors du chargement de widget-settings :
Can't get a way to read the property "affected" in class "Proxies\\__CG__\\App\\Entity\\Partner".

2/ Quand je saisis quelque chose dans le champ Commune ou code postal ou EPCI, ça indique directement Aucun résultat trouvé

3/ Si je saisis quelque chose dans le champ Nom / référence / email, ça prend en compte. Si j'efface le champ, ça ne met pas à jour.

Je pense que j'ai un problème plus global, parce que je n'ai rien dans la liste des étiquettes, et je ne vois pas pourquoi ce serait le cas.
Peut-être le même problème de cache qu'Hélène, mais c'est super bizarre.

@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented May 30, 2024

Décision : on merge en l'état et on créé un ticket pour les retours : #2597

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

Vu ensemble : je merge. Mes retours sont sûrement dus à mon contexte local. A voir plus tard.

@emilschn emilschn merged commit 23c610a into develop May 30, 2024
4 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2121-handle-filter branch June 11, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants