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

Les filtres de tags "complexes" sur les articles ne fonctionnent pas #2674

Closed
SpaceFox opened this issue May 12, 2015 · 9 comments
Closed

Les filtres de tags "complexes" sur les articles ne fonctionnent pas #2674

SpaceFox opened this issue May 12, 2015 · 9 comments
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement !
Milestone

Comments

@SpaceFox
Copy link
Contributor

Aucun des filtres de tags sur les articles qui est plus compliqué que "un seul mot" (exemples : "GNU/Linux", "1er avril", ...) ne fonctionne pas (le filtre n'est pas actif et on voit tout).

Ce n'est pas un bug de la nouvelle version 15.5.1 (il est déjà en prod) mais il devient gênant parce que les tags sont beaucoup plus visibles avec la nouvelle home et les nouvelles pages de tutos / articles.

Tout est OK en ce qui concerne les tags des tutos.

@SpaceFox SpaceFox added S-BUG Corrige un problème C-Back Concerne le back-end Django labels May 12, 2015
@Eskimon
Copy link
Contributor

Eskimon commented May 13, 2015

Faudrait essayer de comparer avec les tutos... Mais sinon coucou les copains de la ZEP-12 ( @artragis et @pierre-24 ) je suppose que chez vous la solution sera bien générique :)

@artragis
Copy link
Member

chez nous les tags des tutos et des articles sont les mêmes donc faudra juste tester.

@Eskimon
Copy link
Contributor

Eskimon commented May 13, 2015

Et j'ai trouve le souci ! C'est "simplement" que l'on passe le nom du tag plutot que son slug pour faire le href ici : https://github.com/zestedesavoir/zds-site/blob/dev/templates/article/includes/article_item.part.html#L76

(du coup c'est vite corrige :) )

@Eskimon Eskimon added the Facile Bon ticket pour débuter pour rejoindre le développement ! label May 13, 2015
@pierre-24
Copy link
Member

Je sais plus si c'est le nom ou le slug que j'ai retenu au final dans la ZEP-12 ;)

@pierre-24
Copy link
Member

... On est bon côté ZEP-12 !

@GerardPaligot
Copy link
Member

Merci à @Eskimon pour le fix et @DevHugo pour la QA.

@firm1 firm1 removed the S-BUG Corrige un problème label May 15, 2015
@firm1
Copy link
Contributor

firm1 commented May 26, 2015

Je réouvre cette issue, car, elle n'est pas corrigée entièrement.

En allant ici ou et en cliquant sur le tag "Gnu/Linux" on tombe sur le bug.

La correction doit aussi être appliquée sur les nouveaux items.

@firm1 firm1 reopened this May 26, 2015
@pierre-24
Copy link
Member

Moui. Sauf que à priori, cette issue est taguée "version de développement", et donc n'est PAS corrigée dans cette bêta ;)

@firm1
Copy link
Contributor

firm1 commented May 26, 2015

En effet, je me suis quelque peu emmêlé les pinceaux entre les versions

Le mar. 26 mai 2015 12:56, Pierre Beaujean notifications@github.com a
écrit :

Moui. Sauf que à priori, cette issue est taguée "version de
développement", et donc n'est PAS corrigée dans cette bêta ;)


Reply to this email directly or view it on GitHub
#2674 (comment)
.

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 Facile Bon ticket pour débuter pour rejoindre le développement !
Projects
None yet
Development

No branches or pull requests

6 participants