Relecture de `vuex-store.md` #17

MachinisteWeb · 2017-09-23T07:33:27Z

Revue de vuex-store.md

Signed-off-by: Bruno Lesieur <[email protected]>

rspt · 2017-09-23T07:50:00Z

en/guide/vuex-store.md

 ---

-> Using a store to manage the state is important to every big application, that's why nuxt.js implement [vuex](https://github.com/vuejs/vuex) in its core.
+> L'utilisation d'un store pour gérer l'état est important pour toutes les applications de taille importante, c'est pourquoi [Vuex](https//vuex.vuejs.org/fr/) est implémenté au cœur de Nuxt.js.


rspt · 2017-09-23T07:51:15Z

en/guide/vuex-store.md

-2. Add `vuex` module in the vendors bundle
-3. Add the `store` option to the root `Vue` instance.
+1. importera Vuex,
+2. ajoutera le module `vuex` dans le packetage vendors,


"paquetage"

Pour aligner notamment avec "empaqueter".

rspt · 2017-09-23T07:52:08Z

en/guide/vuex-store.md

@@ -133,7 +135,7 @@ And in your `pages/todos.vue`, using the `todos` module:
      <input type="checkbox" :checked="todo.done" @change="toggle(todo)">
      <span :class="{ done: todo.done }">{{ todo.text }}</span>
    </li>
-    <li><input placeholder="What needs to be done?" @keyup.enter="addTodo"></li>
+    <li><input placeholder="Qu'est-ce qui dois être fait ?" @keyup.enter="addTodo"></li>


"Qu'est-ce qui doit être fait ?"

rspt · 2017-09-23T07:53:51Z

en/guide/vuex-store.md


-The context is given to `nuxtServerInit` as the 2nd argument, it is the same as the `data` or `fetch` method except that `context.redirect()` and `context.error()` are omitted.
+Le contexte est fourni par `nuxtServerInit` comme 2ème argument. C'est le même que pour les méthodes `data` et `fetch` excepté que `context.redirect()` et `context.error()` sont omis.


"2e argument"

Nous n'avons pas utiliser « 2ᵉ » dans les documentations précédente. Pour plus de consistence je vais écrire deuxième.

Pas de soucis pour ce cas là. Par contre, ce serait bien de changer à l'occasion dans les autres documentations (je peux m'en occuper) pour abréger correctement les adjectifs numéraux ordinaux :)

https://www.google.ch/search?q=abr%C3%A9viation+des+adjectifs+num%C3%A9raux+ordinaux&oq=abbr%C3%A9viation+des+a&aqs=chrome.4.69i57j0l5.8687j0j7&sourceid=chrome&ie=UTF-8

Signed-off-by: Bruno Lesieur <[email protected]>

MachinisteWeb · 2017-09-23T08:39:51Z

Merci @rspt pour tes retours !

forresst

Quelques remarques

forresst · 2017-09-23T09:18:58Z

en/guide/vuex-store.md

-3. Add the `store` option to the root `Vue` instance.
+1. importera Vuex,
+2. ajoutera le module `vuex` dans le paquetage vendors,
+3. ajoutera l'option `store` à l'instance racine de Vue.


Vue => Vue

J'ai déjà corrigé das l'originale. Dès qu'on parle de l'instance racine on se réfère toujours au titre produit :
Source (chercher root) :
— https://vuejs.org/v2/guide/instance.html
— https://vuejs.org/v2/guide/components.html
— ...

forresst · 2017-09-23T09:24:46Z

en/guide/vuex-store.md


- **Classic:** `store/index.js` returns a store instance
- **Modules:** every `.js` file inside the `store` directory is transformed as a [namespaced module](http://vuex.vuejs.org/en/modules.html) (`index` being the root module)
+- **Classique :** `store/index.js` retourne une instance.


store/index.js retourne une instance de store.

forresst · 2017-09-23T09:29:09Z

en/guide/vuex-store.md


-To activate the store with the classic mode, we create the `store/index.js` file which should export a method which returns a Vuex instance:
+Pour activer le store avec le mode classique, nous créons `store/index.js` dans lequel nous exportons l'instance du store :


dans lequel nous exportons l'instance du store : => qui devrait exporter une méthode qui renvoie une instance Vuex :

forresst · 2017-09-23T09:31:59Z

en/guide/vuex-store.md


-> Note: Asynchronous `nuxtServerInit` actions must return a Promise to allow the `nuxt` server to wait on them.
+> Note : Les actions `nuxtServerInit` asynchrones doivent retourner une promesse pour permettre au serveur `nuxt` de les attendres.


attendres => attendre

MachinisteWeb · 2017-09-23T09:33:40Z

Bon pour moi a condition d'uitiliser les caractères utf-8 d'exposition (pas sur la même ligne). Mais il me semble qu'on est raccord car je ne crois pas qu'ils utilisent d'abréviation dans les docs de Vue. (first, second, etc.) [image: Bruno Lesieur] <http://www.lesieur.name/> [image: Logo Google+] <https://plus.google.com/+BrunoLesieur/> [image: Logo Twitter] <https://twitter.com/MachinisteWeb> [image: Logo LinkedIn] <https://www.linkedin.com/in/brunolesieur/> Bruno Lesieur Développeur web Front-end et Node.js Email : [email protected] Site : https://www.lesieur.name/ Le 23 septembre 2017 à 10:59, Julien Grünhagel <[email protected]> a écrit :

…

***@***.**** commented on this pull request. ------------------------------ In en/guide/vuex-store.md <#17 (comment)>: > -The context is given to `nuxtServerInit` as the 2nd argument, it is the same as the `data` or `fetch` method except that `context.redirect()` and `context.error()` are omitted. +Le contexte est fourni par `nuxtServerInit` comme 2ème argument. C'est le même que pour les méthodes `data` et `fetch` excepté que `context.redirect()` et `context.error()` sont omis. Pas de soucis pour ce cas là. Par contre, ce serait bien de changer à l'occasion dans les autres documentations (je peux m'en occuper) pour abréger correctement les adjectifs numéraux ordinaux :) https://www.google.ch/search?q=abr%C3%A9viation+des+ adjectifs+num%C3%A9raux+ordinaux&oq=abbr%C3%A9viation+ des+a&aqs=chrome.4.69i57j0l5.8687j0j7&sourceid=chrome&ie=UTF-8 — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#17 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADs8ubX0q0QJUZAfwXpFZmiK03omjLU2ks5slMiEgaJpZM4PheXH> .

Signed-off-by: Bruno Lesieur <[email protected]>

Revue de vuex-store.md

cadef3b

Signed-off-by: Bruno Lesieur <[email protected]>

MachinisteWeb self-assigned this Sep 23, 2017

MachinisteWeb requested review from forresst, sylvainpolletvillard, atinux and rspt September 23, 2017 07:33

MachinisteWeb mentioned this pull request Sep 23, 2017

État des traductions #1

Open

rspt suggested changes Sep 23, 2017

View reviewed changes

Revue de @rspt

cd85758

Signed-off-by: Bruno Lesieur <[email protected]>

rspt approved these changes Sep 23, 2017

View reviewed changes

forresst requested changes Sep 23, 2017

View reviewed changes

Revue de @forresst

85b5a13

Signed-off-by: Bruno Lesieur <[email protected]>

forresst approved these changes Sep 23, 2017

View reviewed changes

MachinisteWeb merged commit d685338 into working Sep 25, 2017

MachinisteWeb deleted the vuex-store branch September 25, 2017 09:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Relecture de `vuex-store.md` #17

Relecture de `vuex-store.md` #17

MachinisteWeb commented Sep 23, 2017

rspt Sep 23, 2017

rspt Sep 23, 2017

rspt Sep 23, 2017

rspt Sep 23, 2017

MachinisteWeb Sep 23, 2017

rspt Sep 23, 2017

MachinisteWeb commented Sep 23, 2017

forresst left a comment

forresst Sep 23, 2017

MachinisteWeb Sep 23, 2017

forresst Sep 23, 2017

forresst Sep 23, 2017

forresst Sep 23, 2017

MachinisteWeb commented Sep 23, 2017 via email


		The context is given to `nuxtServerInit` as the 2nd argument, it is the same as the `data` or `fetch` method except that `context.redirect()` and `context.error()` are omitted.
		Le contexte est fourni par `nuxtServerInit` comme 2ème argument. C'est le même que pour les méthodes `data` et `fetch` excepté que `context.redirect()` et `context.error()` sont omis.


		To activate the store with the classic mode, we create the `store/index.js` file which should export a method which returns a Vuex instance:
		Pour activer le store avec le mode classique, nous créons `store/index.js` dans lequel nous exportons l'instance du store :


		> Note: Asynchronous `nuxtServerInit` actions must return a Promise to allow the `nuxt` server to wait on them.
		> Note : Les actions `nuxtServerInit` asynchrones doivent retourner une promesse pour permettre au serveur `nuxt` de les attendres.

Relecture de vuex-store.md #17

Relecture de vuex-store.md #17

Conversation

MachinisteWeb commented Sep 23, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MachinisteWeb commented Sep 23, 2017

forresst left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MachinisteWeb commented Sep 23, 2017 via email

Relecture de `vuex-store.md` #17

Relecture de `vuex-store.md` #17