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

Modifier les suggestions de la barre de recherche #3131

Closed
Emeric54 opened this issue Oct 29, 2015 · 15 comments
Closed

Modifier les suggestions de la barre de recherche #3131

Emeric54 opened this issue Oct 29, 2015 · 15 comments
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Milestone

Comments

@Emeric54
Copy link
Contributor

Les suggestions de la barre de recherche restent inchangés depuis des semaines (« Mathématiques, UDK, HTML, Java, PHP ».) Il serait bon qu'elles soient choisis aléatoirement via les différentes catégories du site (voir ici, sujet sur le forum).

@GerardPaligot
Copy link
Member

Est-ce vraiment nécessaire de faire une requête en base supplémentaire rien que pour ça ?

@Emeric54
Copy link
Contributor Author

Je pensais procéder comme pour les citations de Clem, avec un simple fichier texte.

@GerardPaligot
Copy link
Member

Ca serait bizarre parce qu'on aurait des catégories en base de données et dans ce fichier. Il faut alors maintenir ces 2 sources de données. Je ne suis pas contre cette évolution mais faut trouver une meilleure solution.

@GerardPaligot GerardPaligot added Evolution C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Oct 29, 2015
@Emeric54
Copy link
Contributor Author

J'ai repris catégories du post initial sur le forum. Mais l'on peut tout simplement parler de suggestions, pas forcément des catégories du forum. Cela pemettrait de donner simplement un peu de dynamisme.

@SpaceFox
Copy link
Contributor

J'aime bien l'idée. Pour éviter de noyer le serveur avec des requêtes inutiles, on pourrait très bien imaginer mettre cette info en cache et la changer, par exemple, tous les jours - ou même toutes les heures.

@Emeric54
Copy link
Contributor Author

Pourquoi pas si c'est nécessaire. Mais pour des requêtes de ce genre, est-ce que ce l'est vraiment ? (Je n'en ai aucune idée)

@SpaceFox
Copy link
Contributor

Oui et non : un aller-retour à la BDD est en soi peu coûteux, mais ici c'est facile à éviter.

En fait, la requête elle-même est très rapide, mais l'enrobage (préparation, envoi au SGBD, traitement du résultat) est relativement coûteux.

@Emeric54
Copy link
Contributor Author

D'accord. Donc dans ce cas, il serait préférable de mettre l'info en cache (?)

Pour éviter de noyer le serveur avec des requêtes inutiles, on pourrait très bien imaginer mettre cette info en cache et la changer, par exemple, tous les jours - ou même toutes les heures.

Qu'entends tu par cette info ? La liste des catégories ?

(PS : Désolé pour toutes les questions)

@WinXaito
Copy link
Contributor

Qu'entends tu par cette info ? La liste des catégories ?

C'est en effet ce qu'il sous entendait selon moi :) Mais moi je dis pourquoi pas !

@Eskimon
Copy link
Contributor

Eskimon commented Oct 30, 2015

Pourquoi du cache, pourquoi ne peux t-on pas faire exactement comme pour les citations de Clem' qui fonctionne sur ce système ?

@SpaceFox
Copy link
Contributor

Les citations sont prises dans un fichier et sont fixes. L'idée de base, si j'ai bien compris, c'est d'aller chercher les exemples dans les tags existants sur le site.

@Emeric54
Copy link
Contributor Author

Pour mieux prendre un main le code de ZdS, je me suis fait un petit système comme pour les citations de Clem' : 5 suggestions sont choisises aléatoirement dans un fichier texte à chaque chargement de la page. Du coup, ça permet également également d'inclure des suggestions autres que des catégories. Mais le problème est qu'en cas de d'ajout d'une nouvelle catégorie, le fichier ne sera pas automatiquement mis à jour. Mais est-ce vraiment nécessaire ?

@WinXaito
Copy link
Contributor

Au pire on peut imaginer un petit script mis en cron 1 fois par jour qui update le fichier non ?

@Emeric54
Copy link
Contributor Author

Avec la solution actuelle, on est parti sur une liste de suggestions, et non une liste des catégories du site (d'ailleurs, les catégories ne sont pas toutes inscrites).

Je ne sais pas si c'est vraiment nécessaire, l'ajout de nouvelles catégories sur ZdS n'est que peu fréquent.

@pierre-24
Copy link
Member

Voilà :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Projects
None yet
Development

No branches or pull requests

6 participants