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

docs(vuepress): move fr gitbook doc to vuepress #2411

Merged
merged 2 commits into from
Oct 7, 2018

Conversation

Slashgear
Copy link
Contributor

The french doc translation was hidden in the previous gitbook configuration.
This heavy PR moves all the files to the vuepress config.

  • define fr-FR language in conf
  • move gitbook doc in vuepress
  • reshape API page to match vuepress structure

😸 Sorry for the long PR, it is mainly doc duplication

Should the French gitbook documentation be removed ?

@posva
Copy link
Member

posva commented Oct 1, 2018

@haeresis est ce que la doc est toujours a jour?

@MachinisteWeb
Copy link
Contributor

Je ne m'y suis pas penchée depuis que je me suis aperçu qu'elle avait disparue du site officielle. Elle n'a donc pas bouger depuis ce temps.

Je peux mettre le dépot à jour sur Vue-FR pour que quelqu'un se charge de remonter la documentation comme il faut ?

@posva
Copy link
Member

posva commented Oct 2, 2018 via email

@MachinisteWeb
Copy link
Contributor

MachinisteWeb commented Oct 2, 2018

Re @posva, @Slashgear.

Je viens de mettre à jour la doc FR depuis vuejs/vue-router. Elle est à jour sur la branche vuejs-fr/working là ou initialement il y a la version EN.

Les changements de la doc FR qu'il faut vérifier comme ayant été bien pris en compte dans cette PR sont les suivants :

vuejs-fr@c3d78e7#diff-7393f511f340552ce081241ad3956923
vuejs-fr@c3d78e7#diff-3ff19a0c242fe35092695e2b3c139707
vuejs-fr@c3d78e7#diff-dd8d4fb925f18be50fc105c99caa5c39
vuejs-fr@c3d78e7#diff-96c09188202d39327035cdd9d0a997ee
vuejs-fr@c3d78e7#diff-17cb19a97a01cc241a8c5ccefaa42a28
vuejs-fr@c3d78e7#diff-876dd9eb4984814f58a853bb13fed330
vuejs-fr@c3d78e7#diff-08fdbb20fba2653f004ce214c8078366
vuejs-fr@c3d78e7#diff-1f2e0a14b04e95de59687d8c96f71d98
vuejs-fr@c3d78e7#diff-6abb4815dfff12be581e9d3536480d39
vuejs-fr@c3d78e7#diff-11d8910b58f5c70ccb3bedc0f37c892f

De la même manière que pour Vuex, voici comment nous allons procéder quand cette PR sera validée.

Nous allons mettre à jour les branches working et dev depuis la upstream (ici) sur vuejs-fr.
Nous allons reverser le contenu FR dans les contenus EN là ou c'est nécessaire pour la branche working.

Ainsi par la suite, quand la doc EN va bouger, cela créera un conflit dans notre doc EN (qui sera la française), cela nous permettra de traquer tous les endroits ou la doc EN change pour mettre à jour la FR.

Puis, dans notre branche dev, nous remettons le contenu en français de EN dans la partie FR. C'est cette branche dev que nous proposons en PR à la version originale à chaque changement.

Y a t-il des zones d'ombre @Slashgear ?

@Slashgear
Copy link
Contributor Author

@haeresis Je crois que j'ai pris tous les changements. Je vous laisse checker si j'en ai oublié un.

@MachinisteWeb
Copy link
Contributor

Bon pour moi @posva et @Slashgear \o/

Merci !

@posva
Copy link
Member

posva commented Oct 7, 2018

Merci!

@posva posva merged commit 51f7971 into vuejs:dev Oct 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants