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 dates dans les pdf sont en francais apres une migration de la zep-12 #2992

Merged
merged 1 commit into from
Sep 20, 2015
Merged

Les dates dans les pdf sont en francais apres une migration de la zep-12 #2992

merged 1 commit into from
Sep 20, 2015

Conversation

DevHugo
Copy link
Contributor

@DevHugo DevHugo commented Sep 17, 2015

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés artragis#341

Les dates des pdf sont généré en anglais lors de la migration de la ZEP-12. Cette branche résout ce souci.

QA (nécessite les outils latex) :

  • Revenir dans l'historique avant le merge de la ZEP-12.
  • Repartir d'une base propre
  • Executer la commande python manage.py migrate
  • Publier un tutoriel et un article
  • Repasser dans l'historique sur le dernier commit de la branche dev
  • Effectuer la procédure de migration
  • Télécharger le pdf de votre article et de votre tutoriel et vérifier que sur la première page la date est en français

@DevHugo DevHugo changed the title Les dates dans les pdf sont en francais apres une migration Les dates dans les pdf sont en francais apres une migration de la zep-12 Sep 17, 2015
@DevHugo
Copy link
Contributor Author

DevHugo commented Sep 17, 2015

Faudrait que ça passe avant la migration de la prod de la ZEP-12, j'ai besoin d'une QA rapidement.

@Situphen
Copy link
Member

Il faudrait créer une branche release-v15.9 et passer ça en bêta fix non ? Car on a déjà taggé la version 15.9-RC1.

@DevHugo
Copy link
Contributor Author

DevHugo commented Sep 17, 2015

Aie, vous avez déjà tagué. La solution de beta fix me parait bien ! Faudrait déjà voir si ma solution proposé marche avant ! =).

@DevHugo
Copy link
Contributor Author

DevHugo commented Sep 17, 2015

J'ai rebase car j'avais fait une petite connerie dans le code, j'avais oublier de supprimer une ligne.

@pierre-24
Copy link
Member

Je veux bien QA, le problème c'est que j'arrive pas à reproduire le bug à la base, donc c'est chaud. C'est une PR qui doit être testée en bêta, pas moyen autrement ^^

@Situphen
Copy link
Member

Euh, si tu n'arrives pas à reproduire le bug en local, c'est peut-être que le bug est un bug Infra et pas un bug de code ?

@pierre-24
Copy link
Member

C'est ce que je suspecte, ceci dit nos tests sur la bêta n'avait pas non plus donnés grand chose de probant.

@DevHugo
Copy link
Contributor Author

DevHugo commented Sep 17, 2015

J'arrive à reproduire moi, mais il faut faire une migration complète.

Si tu veux juste tester comme ça, tu change dans le fichier markdown (celui dans content-public/nom-contenus/extra-contents/), la date sur la troisième ligne, en une date en anglais (tu peux récupérer un fichier de la beta, si tu le souhaite, la date sera en anglais directement) et tu utilise la commande qui génére le pdf.

En gros, si la date est en anglais dans le fichier markdown dans content-public la date sera en anglais dans le pdf généré, normal, pandoc utilise le fichier markdown pour générer le pdf.

Tu vas me dire pourquoi ça passe si on re-publie, le tutoriel alors ? Tout simplement car Django ne prend pas en compte les settings de la langue dans le render_to_string quand tu utilise depuis la ligne de commande. C'est un problème récurrent sur internet (http://stackoverflow.com/questions/4539366/how-to-set-the-language-in-render-to-string-helper). Par-contre ça marche quand tu appelle la fonction depuis la view. C'est pour-ça que ça marche quand on republie le tutoriel.

@pierre-24
Copy link
Member

Pour moi ça fonctionne. On verra ;)

pierre-24 added a commit that referenced this pull request Sep 20, 2015
Les dates dans les pdf sont en francais apres une migration de la zep-12
@pierre-24 pierre-24 merged commit 475a9c0 into zestedesavoir:dev Sep 20, 2015
@pierre-24 pierre-24 added this to the Version 15.9 milestone Sep 20, 2015
@Situphen Situphen added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Sep 20, 2015
@DevHugo
Copy link
Contributor Author

DevHugo commented Sep 20, 2015

Merci, pour le merge et qui vivra et vera ! au pire, c'est sur la béta ^^.

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-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants