Fix #2176: L'avertissement de reponse entre-temps est mal affiche

[Torejy]

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

QA:

• Tester que le message « Au moins un nouveau message a été posté dans la discussion pendant que vous rédigiez le votre » s'affiche bien.

[landscape-bot]

Code quality remained the same when pulling 99bf2a0 on Torejy:dev into 175eaa6 on zestedesavoir:dev.

[gustavi]

Le saut de ligne est inutile ici

[Torejy]

Ben j'ai fait comme :
là :

zds-site/templates/tutorial/comment/new.html

Line 36 in ba783dc

{% blocktrans %}

là :

zds-site/templates/mp/post/new.html

Line 37 in 159c3a9

<strong>Au moins un nouveau message a été posté</strong> dans la discussion

et là :

zds-site/templates/article/reaction/new.html

Line 30 in 6a85d8e

<strong>Au moins un nouveau message a été posté</strong> dans la discussion

[gustavi]

OK ... Je ne vois pas l’intérêt ! PING @sandhose @Situphen une idée ?

[SpaceFox]

Je mets ça sur le compte de la convention d'indentation des templates du créateur de ces fichiers. Je ne sais pas dans quelle mesure c'est un héritage de Progdupeupl ou non d'ailleurs.

[gustavi]

D'accord :D C'est totalement useless mais pas grave ! Un jour on fera du nettoyage dans les templates !

[SpaceFox]

Le jour où on passera à Jinja2 je suppose :)

[gustavi]

Django 1.8 ;)

[Situphen]

Perso, je trouve que le saut de ligne rend la lecture du template plus simple.

[landscape-bot]

Code quality remained the same when pulling be8cf3b on Torejy:dev into 9870e70 on zestedesavoir:dev.

[coveralls]

Coverage remained the same at 75.49% when pulling be8cf3b on Torejy:dev into 9870e70 on zestedesavoir:dev.

[Torejy]

Ça c'est pas prévu (les trois commits qui se sont rajoutés)
Quelqu'un pourrait me dire ce que je dois faire pour enlever les derniers commits ? J'essaye d'éviter d'accumuler les bétises. :p

[landscape-bot]

Code quality remained the same when pulling 207970a on Torejy:dev into 9870e70 on zestedesavoir:dev.

[SpaceFox]

En première approche, un reset de la branche sur le commit qui va bien, suivi d'un rebase sur dev.

Sauf que visiblement tu as une seule branche pour tes 2 devs. Alors il faudra peut-être repartir de dev, tirer une branche pour chaque développement et faire des cherry pick pour y récupérer les commits qui vont bien.

Si quelqu'un a une idée plus intelligente...

[artragis]

tu peux faire un rebase comme ça :

git rebase -i HEAD~4

dans l'éditeur de texte qui s'ouvre, tu laisses "pickup" pour le premier et le dernier commit et tu replaces pickup par f dans les deux du milieu.

Dans le domaine de la poésie git on appellerait sûrement ça une rime embrassée.

[SpaceFox]

@Torejy tu t'en sors ?

[Eskimon]

Le plus propre/simple serait surement de faire une nouvelle branche/PR:

# on se remet sur dev
git checkout dev
# on cree une branche locale de travail et se met dessus
git checkout -b fix-2176
# on va chercher le commit interessant (un seul si j'ai bien suivi)
git cherry-pick 99bf2a0
# on pousse la branche vers le depot
git push origin fix-2176
# et apres on va refaire la PR :)

[Eskimon]

J'ai refait une PR propre avec juste ton commit unique. Je ferme ici. (J'ai utilisé un cherry-pick donc tu restes auteur du commit ^^ )