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

[Tableaux de bord & Brief] Pouvoir sélectionner toutes les zones reg depuis le filtre + Ajout d'un filtre sur le type de signalement + Classement des zones reg et AMP par ordre alphabétique #2055

Merged
merged 8 commits into from
Jan 30, 2025

Conversation

claire2212
Copy link
Collaborator

@claire2212 claire2212 commented Jan 28, 2025

@claire2212 claire2212 self-assigned this Jan 28, 2025
@claire2212 claire2212 force-pushed the claire/dashabord-improvements branch from 85b78f2 to 9745fd8 Compare January 28, 2025 16:33
Copy link
Collaborator

@maximeperraultdev maximeperraultdev left a comment

Choose a reason for hiding this comment

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

LGTM à une question près :p. Pourquoi ne pas avoir fait le tri en base de données ?

@claire2212 claire2212 force-pushed the claire/dashabord-improvements branch from 9745fd8 to 6cf5230 Compare January 29, 2025 08:32
@maximeperraultdev
Copy link
Collaborator

@claire2212 en navigant dans l'app j'ai remarqué quelques endroits où on ne trie pas les zones reg par ordre alphabétique. C'est surement voulu mais je voulais le noter par acquit de conscience.

  • Dans les zones de vigilances lorsqu'on ajoute des zones en lien :
Capture d’écran 2025-01-29 à 14 20 07 Capture d’écran 2025-01-29 à 14 20 00 * (hors périmètre) Les zones administratives ne sont pas triées Capture d’écran 2025-01-29 à 14 28 38

@claire2212
Copy link
Collaborator Author

@maximeperraultdev je fais les modifs pour les zones liées à une ZV. Pour les zones administratives c'est voulu, c'est des sorte de groupe (toutes les zones de limites, les départements/façades ensemble, etc)

@claire2212 claire2212 force-pushed the claire/dashabord-improvements branch from 0f5576f to d1ef215 Compare January 30, 2025 08:50
Copy link

@claire2212 claire2212 merged commit ca8a484 into main Jan 30, 2025
27 checks passed
@claire2212 claire2212 deleted the claire/dashabord-improvements branch January 30, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants