Rendu de la preview depuis la Pandoc API

[thom4parisot]

• suppression de l'app export
• preview d'un article
• preview d'une version d'article
• preview d'un corpus
• 🐛 Variable "$version" of required type "ID!" was not provided.
• la comptabilité des URL d'annotation est maintenue (peut-être Utiliser dc.identifier et dc.relation.ispartof pour identifier les articles à annoter avec Hypothesis #1180 ?)
• ⚠️ les URLs de preview ne fonctionnent pas si on n'est pas connecté (🤔)
• Ajout de la numérotation des paragraphes en prévisualisation #1001
• tests d'interface

fixes #1178
fixes #1001
fixes #139

[netlify]

✅ Deploy Preview for stylo-docs canceled.

Name	Link
🔨 Latest commit	6838a25
🔍 Latest deploy log	https://app.netlify.com/sites/stylo-docs/deploys/679115f00073f30008d93c22

[netlify]

✅ Deploy Preview for stylo-dev ready!

Name	Link
🔨 Latest commit	6838a25
🔍 Latest deploy log	https://app.netlify.com/sites/stylo-dev/deploys/679115f09f33640008e42a80
😎 Deploy Preview	https://deploy-preview-1179--stylo-dev.netlify.app
📱 Preview on mobile	Toggle 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]

@antoinentl j'ai ajouté les modifications proposées dans #1001. Je m'étonne de ne pas voir les numéros apparaitre :

T'as une petite idée ?

---

@RochDLY @marviro @giuliaferretti12 j'ai bien avancé sur cette bascule. J'étais très motivé et ça n'a pas pris tant de temps que ça. La seule interrogation majeure qu'il me reste, c'est l'accès de l'URL à un compte Stylo non connecté (ça ne fonctionne pas ; car les appels à l'API GraphQL demandent à être authentifié·e).

On maintient le fonctionnement précédent (URL de preview accessibles en libre accès) ou on sécurise (et on créera des URL de partage distinctes des URL d'articles/corpus) ?

[antoinentl]

Merci @thom4parisot ! De mon côté je ne parviens pas à afficher la preview, voici le message d'erreur : "La génération de la prévisualisation a échoué."

[thom4parisot]

@antoinentl tu utilises stylo-export et pandoc-api dans leur commit le plus récent de la branche main ?

# pour https://gitlab.huma-num.fr/ecrinum/stylo/stylo-export
docker buildx build --tag davidbgk/stylo-export:1.1.0 .

# pour https://gitlab.huma-num.fr/ecrinum/stylo/pandoc-api
docker buildx build --tag davidbgk/pandoc-api:0.0.17 .

[antoinentl]

@thom4parisot Oups je pensais que la version de dev déployée (https://deploy-preview-1179--stylo-dev.netlify.app/) me permettrait de tester. Je ne vais pas tester en local (pour comprendre pourquoi la numérotation ne fonctionne pas) donc je laisse d'autres prendre le relais sur la question (@RochDLY ?).

[RochDLY]

@thom4parisot je rebondis juste sur la génération de l'accès à la preview si l'auteur est déconnecté : c'est hyper important qu'on garde ce fonctionnement car c'est par cette page (avec hypothesis) qu'on peut travailler en asynchrone sur un document (ex : un prof qui relie les travaux de ces élèves ; des relecteurs qui annotent un article en cours d'évaluation)

[thom4parisot]

Merci @RochDLY ; j'entends qu'on souhaite garder la possibilité d'annoter un article sans avoir de compte Stylo / sans être authentifié.

Quand on ajoutera #1069, ça ne sera pas vraiment un frein : toute personne avec un compte Hypothesis sera en mesure d'utiliser Stylo. Mais ça oblige quand même à avoir un compte Hypothesis.

Je pensais qu'on avait une issue pour ça mais voici ce que je pense être la trajectoire la plus safe :

1. disposer d'une URL de partage unique (à voir si on peut réutiliser le mécanisme de permissions d'article) ; cette URL contient un jeton permettant l'accès uniquement à cet article en lecture seule (aka la preview)
2. supprimer / renouveler l'URL de partage
3. pouvoir créer / révoquer plusieurs URL de partage avec permissions (lecture, annotation, écriture, export)

On a ainsi une vraie fonction de partage, sécurisée et consentie (aujourd'hui il suffit de connaître l'identifiant d'un article pour y accéder en preview).

@ggrossetie @maiwann vous en pensez quoi ?

[maiwann]

Est-ce que vous pensez qu'il faut une permission spécifique pour l'export ?

[thom4parisot]

En discutant avec @RochDLY toute à l'heure, on s'est dit que le plus petit pas pour commencer c'était de pouvoir interroger l'API de récup des articles (et des corpus) sans vérification d'utilisateur. Au final ça fait comme avec les routes "publiques" qu'on avait précédemment.

Ensuite quand on ajoutera des URL de partage avec droits d'accès, là on pourra restreindre à nouveau.

[thom4parisot]

Est-ce que vous pensez qu'il faut une permission spécifique pour l'export ?

@maiwann Je ne sais pas 🤷🏻 ça dépend s'il y a des enjeux différents entre lire un article, l'annoter et en récupérer le contenu. Cf. #1154

[thom4parisot]

Aussi, avec ce changement, on peut envisager d'avoir la preview dans la même interface que Stylo (avec l'entête, etc.). Et donc basculer dynamiquement de la preview à l'écriture, etc.

[thom4parisot]

Avec le dernier commit (b0a9cd99bd36), on peut avoir les annotations d'un même article sur deux environnements différents (tant que les articles ont le même identifiant).

sur ma machine

sur stylo-dev

[thom4parisot]

Le seul "bug" c'est que les entêtes de la preview ne contiennent pas les métadonnées 🤔