Amélioration de l'autocomplétion

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir

https://zestedesavoir.com

Other

268 stars 161 forks source link

Amélioration de l'autocomplétion #6645

Closed Situphen closed 2 months ago

Situphen commented 2 months ago

Fixes #6460

QA :

source zdsenv/bin/activate && make update && make zmd-start && make run-back
Vérifier le bon fonctionnement de l'autocomplétion dans les MP (ajout d'un destinataire)
Vérifier le bon fonctionnement de l'autocomplétion dans les contenus (ajout d'un auteur ou d'un contributeur)

Idéalement, il faudrait mettre cette PR sur la bêta pour tester avec des vrais noms d'utilisateurs, mais en local on voit clairement une amélioration par rapport à avant je trouve.

coveralls commented 2 months ago

coverage: 88.928% (+0.002%) from 88.926% when pulling 3df9934f0fd4ebaa10ca22cf8c03d541e1ae5dd0 on Situphen:autocomplete into 75989e6a27431451effbedb725fb0ff2cbc07f4b on zestedesavoir:dev.

philippemilink commented 2 months ago

Est-ce que tu pourrais ajouter / compléter des tests, stp ? (tu peux me dire que tu n'as pas envie de le faire, je pourrais le faire dans ce cas)

Situphen commented 2 months ago

J'ai ajouté un petit test pour vérifier que les résultats retournés sont dans le bon ordre, qui est le suivant :

"Est égal"
"Est égal" insensible à la casse
"Commence par"
"Commence par" insensible à la casse
"Contient"
"Contient" insensible à la casse

Le test vérifie aussi que :

les pseudos qui contiennent les lettres du mot ("a", "n", "d" et "r") mais pas le mot recherché ("andr") ne sont pas retournés ;
les pseudos qui contiennent des lettres accentuées peuvent être retournés.