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] Améliorations UI sur les filtres #2699

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#2684
#2683
#2695
#2693

Description

2684 : Dans les filtres, les 2 filtres avec sélection multiple doivent avoir un fond gris
2683 : Pour le filtre étiquette, afficher les étiquettes par ordre alphabétique
2693 : Pour le filtre partenaires, afficher les partenaires par ordre alphabétique
2695 : Dans le filtre EPCI, il faudrait déplacer le numéro insee derrière le nom de l'EPCI et (entre parenthèses)

Changements apportés

  • Changement de l'app Vue

Pré-requis

npm run watch
make clear-pool pool="--all"
make console app="load-epci"

Tests

  • Afficher la nouvelle liste de signalements et vérifier les 4 points ci-dessus
  • Faire des tests de non régression sur les filtres

Copy link

sonarcloud bot commented Jun 14, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

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

See analysis details on SonarCloud

for (const id in requestResponse.partners) {
const partnersArray = Object.values(requestResponse.partners)
partnersArray.sort((a: any, b:any) => (a.nom > b.nom) ? 1 : ((b.nom > a.nom) ? -1 : 0))
partnersArray.forEach((partner: any) => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

J'ai d'abord essayé de faire le tri alphabétique directement dans le repository (pour les étiquettes et pour le partenaire). Mais dans ces 2 requêtes il y a un "indexBy" qui va à l'encontre d'un orderBy. Et cet indexBy est indispensable, notamment dans les filtres de l'ancienne lsite et de la carto. J'ai aussi essayé de faire des nouvelles fonctions dans les repository pour ne pas faire les même requête, sauf que dans la route widget_settings, je n'arrive pas à savoir si je viens de la nouvelle liste ou pas... Bref, après avoir essayé plusieurs choses, j'ai fini par me résoudre à refaire un tri en js en attendant qu'on se débarrasse des anciens filtres

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

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.

Ok

@numew numew merged commit 13496ca into develop Jun 17, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/2684-2683-2695-2693-bo-liste-des-signalements branch June 17, 2024 15:32
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

3 participants