-
Notifications
You must be signed in to change notification settings - Fork 161
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
Fix #736: Recherche auto lorsqu'on crée un sujet #2318
Conversation
Field('subtitle', autocomplete='off'), | ||
HTML(u'<div id="topic-suggest" style="display:none;"><label>{0}</label>' | ||
'<div id="topic-result-container" data-neither="{1}"></div></div>' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Erreur PEP-8 : ./zds/forum/forms.py:55:19: E127 continuation line over-indented for visual indent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
decale la deuxieme chaine d'un espace a gauche ou simplement fait la commencer par un 'u' pour que tout soit aligne verticalement avec la parenthese si je dis pas de betise
Aucune personne pour QA ? |
Bah c'est loin d'etre simple a QA (il faut solr et une base de donnees correcte) et c'est pas urgent compare au reste non plus. |
Je vais QA cette branche. |
Alors, d'un point de vue purement technique, ça fait le boulot. Maintenant, avant de merger, j'aimerais qu'on corrige les points suivants :
Note que je n'ai pas regardé le code. PS : La vrai difficulté de QA ici, c'est les données. Solr s'installe en très peu temps, la seule chose un peu longue c'est le téléchargement de l'archive (160 Mo), le reste il suffit de suivre les instructions et ça marche. |
Quelques problèmes au niveau de l'indentation de ton code ; faudrait passer les tabs en 4 espaces |
Ne serait-t-il pas plus intéressant d'attendre l'API sur le module forum pour ce genre de fonctionnalité ? |
|
La fonctionnalité est intéressante mais je ne suis pas convaincu de son implémentation. Je pense qu'il faut vraiment attendre l'API. |
+1 pour Andr0 Le mer. 11 mars 2015 10:04, Gérard Paligot notifications@github.com a
|
Etant donné que la fonctionnalité existe déjà coté back et que cette PR ne fait que l'utiliser coté front, je ne pense pas qu'il faille forcément attendre l'API pour ça. |
D'après la QA de @SpaceFox, la fonctionnalité implémentée côté back n'est pas satisfaisante. |
Pour être tout à fait honnête, je ne sais pas si ça vient de la fonctionnalité ou de la qualité des résultats renvoyés par Solr. Mais d'un point de vue utilisateur, ce n'est pas vraiment utilisable. |
Si j'en crois le code, les critères sont, pour un titre de topic saisi :
@SpaceFox ce n'est pas ce que tu as observé en QA ? |
Je n'ai pas observé du code mais un fonctionnement. Et le fonctionnement Le 11 mars 2015 10:52, firm1 notifications@github.com a écrit :
|
Disons que si pendant ta QA tu n'avais pas assez de topic résolus, forcément les résultats était inutilisable. Donc c'est important de savoir comment tu as testé et quel résultat non pertinent tu as obtenu par exemple. |
La base que j'ai, c'est un dump anonymisé de la prod. Donc niveau qualité des résultats, on est assez représentatifs je trouve :) Je donnerai un exemple ce soir. |
Ce soir, c'était il y a 4 jours. Des nouvelles @SpaceFox ? |
J'ai pas eu le temps et je ne suis pas chez moi là. |
Je pense qu'il faut une grande population pour pouvoir trouver un sujet correspondant à notre création. (Sauf si on génère volontairement un sujet "identique"). |
Je confirme, avec un dump de la prod anonymisé (PS : et un index Solr propre) qui date de début février, que les résultats sont à peu près inexploitables.
La recherche proposée (les sujets résolus uniquement) marche pas mal quand tu as la chance de tomber directement sur le bon topic et que tu t'amuses à taper des mot-clef dans le champ titre. Hors de ce cas particulier, ça ne fonctionne pas. Je n'ai pas d'idée pour améliorer le fonctionnement là tout de suite, si ce n'est de chercher aussi dans les tutos. |
Je ne suis pas capable de faire le code back. |
|
Vu l'état de la PR, je la déclare abandonnée et donc la ferme. |
Cette fonctionnalité affiche la liste des sujets résolus similaire. La recherche se lance lorsque la zone de titre perd le focus, ou après 2 secondes d'inactivités (si moins de 5 lettres, on peut atteindre 10 secondes, 4L = 4 sec, 3L = 6 sec, 2L = 8 sec, 1L = 10 sec).
La recherche est bridé à une recherche simultané. Quand il n'y a pas de résultat un message s'affiche, et disparait après 6 secondes. Une animation de 0,8 seconde est présente.