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

Change md export #5117

Merged
merged 6 commits into from
Dec 11, 2018
Merged

Change md export #5117

merged 6 commits into from
Dec 11, 2018

Conversation

artragis
Copy link
Member

@artragis artragis commented Nov 27, 2018

Cette PR a pour but de se concentrer sur les exports de contenus, mais à la marge.

Sur le forum, plusieurs bugs ont été détecté :

  • pas d'url latex
  • lorsqu'on a des # dans des blocs de code, on voit apparaître des ## devant
  • le pdf est long à générer (@pierre-24 j'ai mis un draftmode)
  • dans le md (et donc le pdf) l'intro a une partie pour elle seule.

pour la QA :

  • assurez-vous d'avoir la stack complète (install-full) sinon ni le latex ni le pdf ne seront créés

  • créez un tutoriel avec un code C qui a un # include <wtf.io> par exemple

  • publiez-le

  • assurez-vous que dans l'interface le lien de téléchargement latex mène bien audit latex

  • téléchargez le mardown (nécessite d'être staff) => assurez vous que # include est bine mis comme ça et n'est pas précédé d'autre #

@coveralls
Copy link

coveralls commented Nov 27, 2018

Coverage Status

Coverage increased (+33.6%) to 89.25% when pulling d0c7ee7 on artragis:change_md_export into 7e454ce on zestedesavoir:dev.

@artragis artragis added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Dec 4, 2018
@artragis artragis mentioned this pull request Dec 4, 2018
8 tasks
@Situphen
Copy link
Member

Situphen commented Dec 10, 2018

QA OK pour moi :)
sauf pour le draftmode que je ne sais pas comment tester, mais le code à l'air correct

Il faut juste que tu rebases et on peut merger

@artragis
Copy link
Member Author

Done.

@Situphen Situphen merged commit 822d49d into zestedesavoir:dev Dec 11, 2018
@artragis artragis deleted the change_md_export branch April 25, 2021 15:04
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