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

Ajout des tops tags dans la top bar #1386

Merged
merged 2 commits into from Aug 27, 2014
Merged

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Aug 19, 2014

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? oui
Tickets concernés Aucun

Cette PR a pour objectif, afin d'augmenter légèrement la visibilité des tags, et en même temps de combler le vide actuel dans la barre du haut, de rajouter dans la barre du haut, une colonne "Tag les plus utilisés" qui liste l'ensemble des tags du site les plus utilisés.

Le nombre de top tag est paramétrable dans le fichier settings.py (je l'ai indiqué à 5 ici) et aussi surchargeable en fonction de l'activité dans le settings_prod.py en production.

Note pour QA

  • Créez des topics avec un ou plusieurs tags
  • Vérifiez que les 5 tags affichés dans la barre du haut sont bien les tags avec le plus de sujet
  • Vérifiez qu'un tag appartenant à un topic privé uniquement n'est pas visible par un membre qui n'a pas accès à ce topic.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) when pulling f50d442 on firm1:feature-top-tags into 914da02 on zestedesavoir:dev.

.distinct()\
.filter(forum__in=forums, tags__isnull=False)

#for tg in tgs:
Copy link
Contributor

Choose a reason for hiding this comment

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

C'est volontaire c'est commentaires ?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) when pulling 3fe8af5 on firm1:feature-top-tags into 914da02 on zestedesavoir:dev.

@pierre-24
Copy link
Member

Rapport de QA :

  • Tags dans la top bar : OK (m'as fallu le temps de voir de quoi tu parlais ^^)
  • Tags d'un topic privé n'apparaissant pas : OK
  • Par contre, on dirais que t'as changé la page d'accueil des forums. Du coup, j'ai un titre "categorie" (sans accents ni majuscule) et un titre "tags" (sans majuscule) qui sont apparu, j'imagine que c'est fait pour. Par contre dans "categories", les liens ne fonctionnent plus, il renvoient tous vers /forums/, et non plus aux catégories respectives ... pas OK.
  • Note qui sort un peu du contexte de la QA : comme c'est amené à se retrouver sur toutes les pages, un peu de caching serait le bienvenu.Bon, c'est pas non plus la (les?) requête(s) la (les?) plus coûteuse(s) du paquet (j'en compte deux, normalement).

EDIT: j'ai rien dit, la première des deux, c'est les "top-catégories" (je ne vois pas ce que c'est non plus, d'ailleurs ^^ )

@firm1
Copy link
Contributor Author

firm1 commented Aug 24, 2014

Par contre, on dirais que t'as changé la page d'accueil des forums. Du coup, j'ai un titre "categorie" (sans accents ni majuscule) et un titre "tags" (sans majuscule) qui sont apparu, j'imagine que c'est fait pour.

Le but n'était pas de changer cette page. C'est corrigé désormais.

Note qui sort un peu du contexte de la QA : comme c'est amené à se retrouver sur toutes les pages, un peu de caching serait le bienvenu [...]

c'est pour ça qu'on a une issue pour la cache. Et dans notre cas, ça ne coûte pas grand chose.

En bref,

QA CAN CONTINUE

@pierre-24
Copy link
Member

La QA est bonne, à merger :)

Eskimon added a commit that referenced this pull request Aug 27, 2014
Ajout des tops tags dans la top bar
@Eskimon Eskimon merged commit 9581825 into zestedesavoir:dev Aug 27, 2014
@SpaceFox SpaceFox added this to the Version 1.1 milestone Sep 3, 2014
@SpaceFox
Copy link
Contributor

SpaceFox commented Sep 3, 2014

Pour avoir une trace propre, on pourrait créer le ticket qui correspond à cette PR ?

@SpaceFox SpaceFox modified the milestone: Version 1.1 Sep 24, 2014
@firm1 firm1 deleted the feature-top-tags branch June 24, 2015 09:06
@firm1 firm1 restored the feature-top-tags branch March 2, 2018 17:35
@firm1 firm1 deleted the feature-top-tags branch November 5, 2019 08:48
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

Successfully merging this pull request may close these issues.

None yet

6 participants