rte-antares-rpackage / manipulateWidget

Add More Interactivity to htmlwidgets
128 stars 12 forks source link

Create a site with the documentation of the package #18

Open FrancoisGuillem opened 7 years ago

cderv commented 7 years ago

A first version seems to work. Currently on cderv/manipulateWidget@docsite (could make a PR to work on it if you prefer) However, several issues need to be delt with:

FrancoisGuillem commented 7 years ago

1- This is strange. Have you tried to update your packages? 2- 3- OK for NEWS.md 4- You are talking about the examples in the "reference" section? 5- I agree. README should be shorter and have a link to the package vignette.

cderv commented 7 years ago

Yes I was talking about examples in "reference" section. They are not run because they produce a widget. I consider it is ok not to run them as you give examples of usage in vignettes.

Readme should be shorter we agree and it sould have a link to website when it will be ready, rather than the vignette. We could discuss Readme content in another issue if you want.

FrancoisGuillem commented 7 years ago

Je suis d'accord pour les exemples, c'est pas grave si on peut pas les exécuter. Et pour tout le reste, je suis OK. Quand tu as fini de corriger les petits soucis, tu peux créer une PR avec le README et la vignette actuels ? Je les ai pas mal modifiés sur ma branche, donc je m'occuperai de les découper correctement.

FrancoisGuillem commented 7 years ago

Pour le badge, c'est pas un soucis de proxy ? Quand j'essaie de générer le fichier dans Rstudio, il essaie de télécharger le badge et de le base64-encoder mais il se fait envoyer bouler par le proxy !

cderv commented 7 years ago

Proxy peut être, j'essaierai de nouveau dès que possible. En fait, J'ai une erreur côté pandoc, pour la conversion en html donc.

Pour réintégrer le travail, je peux repasser sur une de tes branches sinon afin de faire un rebase de la mienne et vérifier que ça marche. Sinon je ferais ce que tu me demandes.

FrancoisGuillem commented 7 years ago

C'est comme tu le sens. C'est la branche develop.

cderv commented 7 years ago

Avancement :

Il me reste donc encore à comprendre pourquoi ça marche pas. Dans le but d'avoir une première version, je propose entre temps de créer une page d'accueil du site dédiée indépendante du readme. De cette façon, pas de doublons entre la page d’accueil et la vignette. (ce qui est tjs le cas actuellement Pour cela, il me reste à compléter le readme comme demandé en #19, faire les ajustements et je prépare une PR propre pour intégration.

FrancoisGuillem commented 7 years ago

Ca y est j'ai découpé la vignette et le README. Ils sont disponible dans la branche develop

cderv commented 7 years ago

Impec. Dans mes derniers tests, j'ai toujours besoin dans le README

Pour le reste ça fonctionne bien en compilant le site avec pkgdown::build_site(example = F). J'ai fait un fichier de config _pkgdown.yml pour tuner un peu le site.

Faut que je reparte de ton dernier commit de develop pour recréer une branche ç jour- mais tu peux tester direct chez toi si tu veux entre temps.

cderv commented 7 years ago

Je continue le boulot là dessus. J'ai plus de problème avec les badges, ça fonctionne maintenant. Par contre, comme votre readme.Rmd et readme.md contient des gif aussi utile dans la vignette, ça pose problème car ce n'est pas vraiment compatible par défaut. Je cherche comment faire. Je vais pousser le travail en PR même si ce n'est pas fini pour que ça soit plus simple pour vous de voir à quoi ça ressemble. C'est dans une de mes branches sur le fork pour le moment.