Relecture de `examples/*` by MachinisteWeb · Pull Request #22 · vuejs-fr/nuxt

MachinisteWeb · 2017-10-03T13:39:50Z

Tous les exemples revus

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

rspt · 2017-10-03T13:49:04Z

en/examples/auth-routes.md

 ```

-And we update our `package.json` scripts:
+Et nous modifions nos script dans `package.json` :


"nos scripts"

rspt · 2017-10-03T13:49:25Z

en/examples/auth-routes.md

-3. We create our `SET_USER` mutation which will set the `state.authUser` to the connected user
-4. We export our store instance to Nuxt.js can inject it to our main application
+1. Nous importons `Vue` et `Vuex` (inclus dans Nuxt.js) et nous indiquons à Vue d'utiliser Vuex afin de pouvoir utiliser `$store` dans nos composants.
+2. Nous utilisons `require('whatwg-fetch')` afin de d'obtenur un polyfill pour la méthode `fetch()` pour tous les navigateurs (consultez le [dépôt fetch](https://github.com/github/fetch)).


"afin d'obtenir"

rspt · 2017-10-03T13:50:03Z

en/examples/auth-routes.md


-1. returning a `Promise`, nuxt.js will wait for the promise to be resolved before rendering the component.
-2. Using the [async/await proposal](https://github.com/lukehoban/ecmascript-asyncawait) ([learn more about it](https://zeit.co/blog/async-and-await))
+### L'ation login()


Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

…ples

MachinisteWeb · 2017-10-03T14:01:29Z

Bwarf, pas très réveillé moi aujourd'hui... Merci @rspt !

forresst

Quelques erreurs et remarques. C'est du très bon boulot !!!

forresst · 2017-10-03T14:17:43Z

en/examples/auth-routes.md

 # Documentation

-> Nuxt.js can be used to create authenticated routes easily.
+> Nuxt.js peut être utilisé pour créer des routes authentifiées facilement.


Nuxt.js peut être utilisé pour créer facilement des routes authentifiées ~~facilement~~ .

forresst · 2017-10-03T14:19:50Z

en/examples/auth-routes.md

 })

-// POST /api/logout to log out the user and remove it from the req.session
+// Accès à `/api/logout` en POST pour désauthentifier l'utilisateur et le retirer de `req.session`


désauthentifier => déconnecter

Je mets « connecter » plus haut pour rester cohérant

forresst · 2017-10-03T14:25:16Z

en/examples/auth-routes.md

+## Utilisation du store

-We need a global state to let our application know if the user is connected **across the pages**.
+Nous avons besoin d'un état global pour informer notre application si l'utilisateur est **connecté sur les pages**.


Nous avons besoin d'un état global pour savoir si l'utilisateur est toujours connecté sur les différentes pages de notre application.

"across" dans cette phrase indique le passage d'une page à une autre, donc ne pas perdre la connexion lors du passage d'une page à l'autre. Ma proposition me parait plus en adéquation avec l'original. Ton avis ?

je vais mettre « reste connecté entre les pages » pour coller à across effectivement.

forresst · 2017-10-03T14:27:43Z

en/examples/auth-routes.md

 ```

-*We will talk about `whatwg-fetch` later.*
+*Nous parlerons de `whatwg-fetch` dans un instant.*


Nous parlerons de whatwg-fetch plus tard.

"Dans un instant", cela correspond "à toute de suite" : ce n'est pas le cas

Ça correspond plutôt à d'ici quelque minutes. Ce qui correspond au temps moyen de lecture avant qu'on se penche sur ce truc.

Je vais mettre plus loin sous entendu « plus loin dans le texte ».

forresst · 2017-10-03T14:28:36Z

en/examples/auth-routes.md

+### Fonction nuxtServerInit()

-Nuxt.js will call a specific action called `nuxtServerInit` with the context in argument, so when the app will be loaded, the store will be already filled with some data we can get from the server.
+Nuxt.js appelle une action spécifique nommée `nuxtServerInit` avec le contexte comme argument. Ainsi, lorsque l'application sera chargée, le store sera déjà rempli avec certaines données que nous pouvons obtenir du serveur.


appelle => appellera

forresst · 2017-10-03T14:29:47Z

en/examples/auth-routes.md

 login ({ commit }, { username, password }) {
  return fetch('/api/login', {
-    // Send the client cookies to the server
+    // Envoyer les cookies client au serveur


Envoyer => Envoie

forresst · 2017-10-03T14:30:03Z

en/examples/auth-routes.md

 logout ({ commit }) {
  return fetch('/api/logout', {
-    // Send the client cookies to the server
+    // Envoyer les cookies au serveur


Envoyer => Envoie

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

MachinisteWeb

Bien joué @forresst ! C'est surtout @rspt qui a bien bosser sur la traduction initial ! Quasiment tout est déjà traduit !

MachinisteWeb · 2017-10-03T14:37:49Z

en/examples/auth-routes.md

 ```

-*We will talk about `whatwg-fetch` later.*
+*Nous parlerons de `whatwg-fetch` dans un instant.*


Ça correspond plutôt à d'ici quelque minutes. Ce qui correspond au temps moyen de lecture avant qu'on se penche sur ce truc.

Je vais mettre plus loin sous entendu « plus loin dans le texte ».

MachinisteWeb · 2017-10-03T14:38:26Z

en/examples/auth-routes.md

 })

-// POST /api/logout to log out the user and remove it from the req.session
+// Accès à `/api/logout` en POST pour désauthentifier l'utilisateur et le retirer de `req.session`


Je mets « connecter » plus haut pour rester cohérant

MachinisteWeb · 2017-10-03T14:41:36Z

en/examples/auth-routes.md

+## Utilisation du store

-We need a global state to let our application know if the user is connected **across the pages**.
+Nous avons besoin d'un état global pour informer notre application si l'utilisateur est **connecté sur les pages**.


je vais mettre « reste connecté entre les pages » pour coller à across effectivement.

forresst · 2017-10-03T14:48:17Z

👏 @rspt et 👏 @haeresis

Partie exemple revue

5c7a424

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

MachinisteWeb self-assigned this Oct 3, 2017

MachinisteWeb requested review from Kocal, atinux, forresst and rspt October 3, 2017 13:40

Merge branch 'working' into examples

52b0cdd

MachinisteWeb added the Relecture nécessaire label Oct 3, 2017

MachinisteWeb mentioned this pull request Oct 3, 2017

État des traductions #1

Open

rspt suggested changes Oct 3, 2017

View reviewed changes

MachinisteWeb added 2 commits October 3, 2017 16:00

@rspt review

0fa8d3b

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

Merge branch 'examples' of https://github.com/vuejs-fr/nuxt into exam…

7e90ee6

…ples

rspt approved these changes Oct 3, 2017

View reviewed changes

forresst requested changes Oct 3, 2017

View reviewed changes

@forresst review

a15ff31

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>

MachinisteWeb commented Oct 3, 2017

View reviewed changes

forresst approved these changes Oct 3, 2017

View reviewed changes

MachinisteWeb merged commit aa5a431 into working Oct 3, 2017

MachinisteWeb deleted the examples branch October 3, 2017 15:02

Conversation

MachinisteWeb commented Oct 3, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MachinisteWeb commented Oct 3, 2017

Uh oh!

forresst left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MachinisteWeb left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

forresst commented Oct 3, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants