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

Export des corpus avec la Pandoc API #1174

Merged
merged 10 commits into from
Jan 22, 2025
Merged

Export des corpus avec la Pandoc API #1174

merged 10 commits into from
Jan 22, 2025

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Dec 31, 2024

Export d'article

image

Export de corpus

image


  • mémorise les choix
  • affiche des choix liés à link-citations et nocite
  • sections et chapitres : numérotés / non numérotés
  • division du corpus : parties et chapitres / chapitres seulement
  • utiliser des images Docker actualisés de stylo-export et de pandoc-api (essais effectués avec la branche main)

fixes #1067
fixes #929
fixes #1121

Copy link

netlify bot commented Dec 31, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit a5ea1d5
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/6790f4526d67d000085abe12

Copy link

netlify bot commented Dec 31, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit a5ea1d5
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/6790f4524cff5d0007118d2c
😎 Deploy Preview https://deploy-preview-1174--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@thom4parisot thom4parisot force-pushed the feat/1067 branch 2 times, most recently from 7677ce5 to a37c120 Compare January 7, 2025 14:52
@thom4parisot thom4parisot marked this pull request as ready for review January 7, 2025 17:00
@thom4parisot thom4parisot force-pushed the feat/1067 branch 2 times, most recently from 80793a5 to b317561 Compare January 21, 2025 10:59
@@ -160,7 +165,10 @@ export default function Export({
</Select>
)}
{bookId && (
<Select value={tld} onChange={(e) => setTld(e.target.value)}>
<Select
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davidbgk @RochDLY je me rends compte qu'il y a deux options d'export spécifique aux corpus :

  • unnumbered : sections et chapitres numérotées ou non
  • book_division : division en chapitres ou en parties

Ce sont des paramètres à passer aux templates ? Ou c'est une option pandoc ?

Ces options sont proposées dans l'interface mais ça restera sans effet pour l'instant.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah tiens je n'ai jamais croisé ces options de manières explicite côté export 🤔

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marviro ça te parle ces options ? Elles étaient envoyés spécifiquement à ton script exportBook/exec.cgi.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oui: importantes pour l'export book, en effet

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vous en avez parlé avec @davidbgk ? Parce qu'avec cette seule info, j'ai du mal à savoir où ces options agissent. Je n'en trouve pas de trace dans les templates de l'export.

Copy link
Collaborator

@RochDLY RochDLY Jan 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est une option qu'on avait pour les exports "book" qui ont été débranchés. On ne les utilise pas pour l'export corpus actuel (plus centré sur les numéros de revue). Par contre on en aura besoin pour générer des livres et/ou des thèses (avec LaTeX) : de mémoire ces options ne servent que pour produire le tex et le pdf

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK : on les enlève (temporairement) de l'interface vu qu'il n'y a pas d'utilité pour le moment ? Ou on les garde et implémente leur fonctionnement plus tard ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A première vue, je dirai qu'on enlève pour le moment ! Idéalement ces paramètres ne devraient apparaitre que lorsqu'on sélectionne du TeX et/ou du PDF. @marviro t'en dis quoi ?

@ggrossetie
Copy link
Collaborator

Quelques remarques (à valider par @maiwann) :

  • Est-ce qu'il ne serait pas préférable d'avoir en label "Section et chapitres" et "Division du corpus" ?

    image

  • Est-ce que pour "Lier les citations" on ne ferait pas plutôt une checkbox ?

  • Même si c'est un peu en dehors du périmètre "stylo", est-ce qu'il ne faudrait pas utiliser la même terminologie sur https://export.stylo-dev.huma-num.fr/

    image

@ggrossetie
Copy link
Collaborator

Comme c'est le dernier point qui bloque la mise en production (et que @maiwann ne sera disponible que semaine prochaine), est-ce qu'on fusionne en l'état et on fera les changements nécessaires dans un second temps ? Est-ce qu'on scinde les changements afin de résoudre uniquement #1121 et garder les autres sujets (comme l'export des corpus avec la Pandoc API) pour plus tard ?

Un avis @RochDLY @thom4parisot ?

@RochDLY
Copy link
Collaborator

RochDLY commented Jan 22, 2025

Comme c'est le dernier point qui bloque la mise en production (et que @maiwann ne sera disponible que semaine prochaine), est-ce qu'on fusionne en l'état et on fera les changements nécessaires dans un second temps ? Est-ce qu'on scinde les changements afin de résoudre uniquement #1121 et garder les autres sujets (comme l'export des corpus avec la Pandoc API) pour plus tard ?

Un avis @RochDLY @thom4parisot ?

Quand tu parles de l'export des corpus, tu parles bien des options pour les tex/pdf (section & chapitre + division du corpus) ?
Ces deux options on peut effectivement les laisser pour plus tard, on a pas les exports prêts pour ça de toute façon !

Ça me va de scinder le problème. Pour ces deux options, on pourra les résoudre dans un second temps !

@thom4parisot thom4parisot merged commit 32a438b into master Jan 22, 2025
14 checks passed
@thom4parisot thom4parisot deleted the feat/1067 branch January 22, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants