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

[bêta v1.8] Fixe erreurs unicodes #2607

Merged
merged 1 commit into from
May 1, 2015

Conversation

pierre-24
Copy link
Member

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #2584 #2606 #2585

Pourquoi, comment ?

Parce que la PR #2585 fait exactement la même chose, mais sur la version de prod'. Sauf que sur la version de prod', c'est toujours django 1.6 et que je tombe sur une erreur mystique (du genre que j'ai passé deux heure a essayer de la fixer). D'autant que #2588 règle #2585 ... De la mauvaise manière (en faisant disparaître le bug dans le markdown, donc à la source). Donc je fixe les problèmes convenablement sur la bêta (avec Django 1.7).

Pourquoi il n'y a pas de T.U.: parce que tant que #2588 est pas mergée, le moindre test que je pourrait mettre en place plantera lamentablement, puisque markdown nous lâche sur les liens avec accents, et que c'est justement ce que cette PR corrige.

Est ce que c'est du bugfix ? je pense, oui. C'était déjà du hotfix à la base, donc bon.

Note de QA

je préviens, c'est pas une QA drôle

  • Commencez avec la version de markdown actuelement en bêta (pip install --upgrade https://github.com/zestedesavoir/Python-ZMarkdown/archive/2.6.0-zds.5.zip). C'est le seul moyen d'avoir un bug markdown et que cette PR soit testable, donc faites le.
  • Créez un article et, pour texte, écrivez :
Cette image va planter: ![](http://upload.wikimedia.org/wikipedia/commons/thumb/c/c6/Latin_alphabet_Ćć.svg/langfr-260px-Latin_alphabet_Ćć.svg.png)

Puis ce lien va planter, aussi : [ce lien va rater](http://fr.wikipedia.org/wiki/Clémentine) 
  • Tentez quand même de valider l'article et vérifiez que vous êtes bien arrêté avant la fin par un message d'erreur qui vous dit que quelque chose ne va pas dans le code markdown.
  • Faites de même avec un tutoriel, mais cette fois, vérifiez que vous êtes également empêché de publier si l'erreur se trouve dans une introduction/conclusion

PUIS !

  • Mettez à jour votre version de markdown (pip install --upgrade https://github.com/zestedesavoir/Python-ZMarkdown/archive/2.6.0-zds.7.zip, ça vient de [hotfix] Correction du bug unicode dans le markdown #2588)
  • Vérifiez que cette fois, tuto et article passent la validation ET que l'image est bien affichée.

Merci à @GerardPaligot pour son acharnement sur la première PR, injustement remercié par mon comportement d'il y a quelques heures.

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling 6b13737 on pierre-24:bugfix_unicode into 8e02a54 on zestedesavoir:release-v1.8.

@pierre-24 pierre-24 changed the title Fixe erreurs unicodes [bêta v1.8] Fixe erreurs unicodes Apr 30, 2015
@GerardPaligot
Copy link
Member

Je me charge de ta QA ce soir. :)

Merci à @GerardPaligot pour son acharnement sur la première PR, injustement remercié par mon comportement d'il y a quelques heures.

Aucun souci.

@Situphen Situphen added C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Apr 30, 2015
@DevHugo
Copy link
Contributor

DevHugo commented Apr 30, 2015

Qa: ok.

Eskimon added a commit that referenced this pull request May 1, 2015
[bêta v1.8] Fixe erreurs unicodes
@Eskimon Eskimon merged commit d2ffdb3 into zestedesavoir:release-v1.8 May 1, 2015
@Eskimon
Copy link
Contributor

Eskimon commented May 1, 2015

YaY bravo Pierre !! et merci Hugo pour la QA !

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 S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants