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] Comportement filtre "signalements importés" + Divers améliorations #2710

Merged
merged 8 commits into from
Jun 20, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Jun 18, 2024

Ticket

#2667
#2666
Régression #2695

Description

Plusieurs améliorations pour optimiser l'usage de la liste et des filtres.

  • La procédure constatée suite à une visite est maintenant affichée au lieu de la procédure suspectée.
  • Le fonctionnement du badge "Afficher les signalements importés" a été revu : les signalements importés ne s'affichent plus par défaut mais deviennent visibles en activant ce badge.
  • La gestion du bouton retour a été effectuée pour conserver les filtres appliqué.
  • Régression corrigé sur l'affichage des badges EPCI avec un affichage correct du badge avec le code approprié dans l'URL.

Changements apportés

  • Gestion de l'affichage EPCI et de la conservation des filtres via le localStorage
  • Suppression de hello_controller.js
  • Mise à jour des composants
  • Mise à jour du service de filtre pour le changement de comportement du filtre sur les signalement importé
  • Mise à jour d'une requête pour récupérer les conclusions de visites

Pré-requis

make npm-watch

Les données de liste déroulante provenant de la base de données sont en cache, si besoin

 make clear-pool pool="--all" make clear-pool pool="--all"

Si besoin de toutes les EPCIs (il y'en as quelque uns dans les fixtures)

make console app="load-epci"

Tests

  • Afficher la liste et vérifier que les signalements importés ne s'affichent pas par défaut.
  • Cliquer sur le badge Afficher mes affectations uniquement et vérifier que les signalements importés s'affichent.
  • Afficher sur la carte la conclusion de visite (Procédure constatée) si renseignée, au lieu de la qualification (Procédure suspectée).
  • Ajouter une visite avec conclusion de visite, la carte doit se mettre à jour avec la dernière conclusion (date de visite)
  • Ajouter des filtres et se rendre sur une fiche de signalement, puis revenir en arrière ; vous devez conserver la liste avec les filtres appliqués.
  • Filtrer par EPCI et vérifier l'URL (seul le code doit figurer) et vérifier que le badge s'affiche correctement.

@sfinx13 sfinx13 force-pushed the bugfix/2667-comportement-importe branch from b32cfb7 to 97f4cbc Compare June 18, 2024 18:48
@sfinx13 sfinx13 requested review from emilschn, hmeneuvrier and numew and removed request for hmeneuvrier June 19, 2024 07:30
@hmeneuvrier
Copy link
Collaborator

Tests OK, pas encore relu

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.

Test et relecture ok.
J'approuve malgré le petit détail suivant : en super admin si je sélectionne un territoire sans signalements importés le bouton pour les affichés disparaît et il ne réapparaît quand je quitte le territoire (en cliquant sur le badge)

@@ -59,8 +59,13 @@ const displayPhotoAlbum = (photoId) => {

document?.querySelector('.back-link')?.addEventListener('click', (event) => {
event.preventDefault()
if (window.history.length === 1) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

dans ce cas, tu n'as qu'à pas faire le preventDefault, non ?
Ca simplifie un peu le code :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

pour éviter les effets de bord vaut mieux tout gérer en JS le href je le laisse pour la forme

Copy link

sonarcloud bot commented Jun 20, 2024

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.

OK lecture et tests !

@emilschn emilschn merged commit 90d894f into develop Jun 20, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/2667-comportement-importe branch June 20, 2024 16:03
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.

4 participants