Messages d'information erronés au sujet d'un article

[orandin]

Q	R
Type de modification	Correction de bug
Ticket(s) (issue(s)) concerné(s)	#3433

Sur certaines actions réalisées sur un article, les notifications évoquaient un "tutoriel" au lieu "d'un article". Cette PR corrige ce défaut de traduction.

QA :

• Sur un article non validé & vide :
  • Vérifier le message évoquant que l'article n'a pas encore de validateur.
  • Vérifier le message évoquant que l'article est vide.
  • Mettre en bêta :
    • Vérifier le message de la modale.
  • Publier l'article & le dépublier pour vérifier le message.
    • Attendu : L'article a été dépublié.
• Sur un tutoriel non validé & vide :
  • Reprendre chaque étape pour un article. Là où l'on pouvait lire "article", on doit lire "tutoriel".

---

Rendu :

Dé-publication

Attente de validation

Article vide

Mettre un article en bêta

Article en cours de validation

[firm1]

@orandin dans les autres templates on utilise le terme générique "contenus" à la place de "tutoriel" et/ou "article". Je pense que tu peux garder cette logique, ça permet de nous éviter pas mal de if/else et si demain on rajoute un nouveau type de contenus, on reste couvert.

[orandin]

Et hop, voilà avec le terme générique. :)

[artragis]

peux tu faire un squash/fixup s'il te plait? Comme ça on pourra merger.

[orandin]

@artragis : fixup/reword fait ! :)

[coveralls]

Coverage decreased (-15.9%) to 71.92% when pulling 4eb2398 on orandin:fix-3433 into 9a844fb on zestedesavoir:dev.

[gustavi]

La même ici. Travis buggé ou c'est nous ?

[orandin]

Avant de faire le fixup/reword, Travis passait et depuis, le job 2 plante systématiquement au même endroit.

[artragis]

As-tu mis le upstream à jour (git fetch upstream) avant de faire le
rebase? car comme il y a eu le merge de la zep-24 entre deux c'est peut
être ça qui foire.

Le 19/03/2016 14:05, Simon Delberghe a écrit :

Avant de faire le fixup/reword, Travis passait et depuis, le job 2
plante visiblement systématiquement au même endroit.

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

[coveralls]

Coverage remained the same at 87.84% when pulling f54b1c6 on orandin:fix-3433 into 2e98229 on zestedesavoir:dev.

[orandin]

Pointeurs mis à jour pour récupérer le patch (35c2f72) de @artragis, qui corrige l'erreur rencontrée par Travis.

[gustavi]

QA : OK.