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

Amélioration de l'autocomplétion #6645

Merged
merged 5 commits into from
Sep 22, 2024

Conversation

Situphen
Copy link
Member

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
Copy link

coveralls commented Sep 18, 2024

Coverage Status

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

@philippemilink
Copy link
Member

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
Copy link
Member Author

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

  1. "Est égal"
  2. "Est égal" insensible à la casse
  3. "Commence par"
  4. "Commence par" insensible à la casse
  5. "Contient"
  6. "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.

@Situphen Situphen force-pushed the autocomplete branch 10 times, most recently from aef16e7 to 7ee50e4 Compare September 21, 2024 22:51
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Les commentaires à ajouter et je fusionne.

J'ai testé en local, ça fonctionne bien ; et j'ai déployé le patch sur la bêta (par-dessus la PR Typesense) et en effet, ça fonctionne mieux.

assets/js/autocompletion.js Show resolved Hide resolved
zds/member/api/tests.py Outdated Show resolved Hide resolved
@philippemilink philippemilink merged commit b14be34 into zestedesavoir:dev Sep 22, 2024
8 checks passed
@Situphen Situphen deleted the autocomplete branch September 22, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

L'autocomplétion dans les MP ne trouve pas correctement les membres
3 participants