-
Notifications
You must be signed in to change notification settings - Fork 12
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
Create a site with the documentation of the package #18
Comments
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:
|
1- This is strange. Have you tried to update your packages? |
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. |
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. |
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 ! |
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. |
C'est comme tu le sens. C'est la branche develop. |
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. |
Ca y est j'ai découpé la vignette et le README. Ils sont disponible dans la branche develop |
Impec. Dans mes derniers tests, j'ai toujours besoin dans le README
Pour le reste ça fonctionne bien en compilant le site avec 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. |
Je continue le boulot là dessus. J'ai plus de problème avec les badges, ça fonctionne maintenant. |
No description provided.
The text was updated successfully, but these errors were encountered: