Correction affichage du tag

[orandin]

Q	R
Type de modification	correction de bug
Ticket(s) (issue(s)) concerné(s)	#3589

Cette PR a le mérite de corriger le bug, mais je ne suis pas satisfait du fix.

Dans la template, j'appelle trois fois le même filtre join pour avoir systématiquement le même résultat (beurk!). J'ai tenté de définir une variable dans la template, mais en vain. J'ai tenté de mettre mon nez dans le back, mais je me suis vite paumé. ^^

Si quelqu'un a une meilleure idée/piste, je suis preneur !

QA

• Faire une recherche par tag (exemple windows comme dans le ticket)
• Vérifier dans tout le code source de la page l'absence de [<Tag: windows>] (remplacer windows par le tag que vous aurez choisi)

[coveralls]

Coverage remained the same at 87.36% when pulling c5add63 on orandin:fix-3589-v18 into 7d74dc7 on zestedesavoir:release-v18.

[firm1]

Aie. Je ne suis pas fan non plus de la façon donc le problème est corrigé.

Il me semblait qu'il suffit de faire une {{ tag.title }} pour corriger ça.

[artragis]

sinon, tu peux toujours faire

{% captureas tagtitle %}
    {{ tag|join }}
{% endcaptureas %}

comme ça où que tu veuilles, tu peux appeler {{ tagtitle }}.
Mais sinon, c'est effectivement tag.title

[GerardPaligot]

Je confirme que la solution n'est pas bonne et qu'il faut changer pour {{ tag.title }}.

@orandin Tu sais t'en charger rapidement ?

[artragis]

Actuellement, j'ai des problèmes sur mon installation locale, donc non.

Je vais juste me charger de la partie répétition de tag (dans une autre
issue) dèsq ue mon environnement sera d'équerre.

Le 09/05/2016 à 11:17, Gérard Paligot a écrit :

Je confirme que la solution n'est pas bonne et qu'il faut changer pour
|{{ tag.title }}|.

@orandin https://github.com/orandin Tu sais t'en charger rapidement ?

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#3597 (comment)

[GerardPaligot]

@artragis Tu sais que c'est @orandin l'auteur de la PR et que c'est @orandin que je pingais ? :-°

[artragis]

Oups.

Le 09/05/2016 à 11:36, Gérard Paligot a écrit :

@artragis https://github.com/artragis Tu sais que c'est @orandin
https://github.com/orandin l'auteur de la PR et que c'est @orandin
https://github.com/orandin que je pingais ? :-°

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3597 (comment)

[coveralls]

Changes Unknown when pulling d69916b on orandin:fix-3589-v18 into * on zestedesavoir:release-v18*.

[coveralls]

Changes Unknown when pulling d69916b on orandin:fix-3589-v18 into * on zestedesavoir:release-v18*.

[GerardPaligot]

Bien, cette PR ne pourra pas être corrigée tant qu'il y aura plusieurs tags par "title" et/ou par "slug" (cela dépendra du choix que nous ferons).

see #3601

[artragis]

C'est corrigé.

[artragis]

fallait pas fermer gustavi !

[coveralls]

Changes Unknown when pulling d69916b on orandin:fix-3589-v18 into * on zestedesavoir:release-v18*.

[coveralls]

Coverage increased (+0.01%) to 87.409% when pulling c7e68e7 on orandin:fix-3589-v18 into 2b0663f on zestedesavoir:release-v18.

[orandin]

Bon pour Q.A. Je suis cette fois-ci satisfait de mon fix.

[GerardPaligot]

QA ok. :)

[gustavi]

J'avais fait une PR qui corrigeais un dernier détail à @orandin là dessus :/

[GerardPaligot]

Ah ? Pourtant j'ai rencontré aucun problème. Tu peux faire une PR direct sur le repo ?