Update Spanish i18n #88

Josexv1 · 2018-07-27T19:11:40Z

Bump Spanish i18n to the latest version
Delete some redundant files to match with English folder

anonimal · 2018-07-28T23:04:26Z

i18n/es/user_guide.md

 ```bash
-$ cd build/ && ./kovri
+$ cd build / && ./kovri


Won't work. Keep the old line.

Done, don't know how it slip, since i didn't touch the code.

erciccione · 2018-07-29T14:43:05Z

@lh1008 @BlackLotus64 could any of you review this?

BlackLotus64 · 2018-07-29T17:06:59Z

Sure @erciccione, I'll take a look at them, I'll try and coordinate with @lh1008 to do this :)

lh1008 · 2018-07-30T15:56:15Z

Hi @erciccione sure, I'll be looking at it. :)

lh1008 · 2018-07-30T15:59:41Z

@BlackLotus64 have you already made the fixes in the spanish guide?

lh1008

I had already gave some comments to a past translation. I have to go and I will start again later in the review. I suggest this changes done for the i18n/es/acknowledgements.md file.

lh1008 · 2018-07-30T16:12:14Z

i18n/es/acknowledgements.md

@@ -1,3 +1,4 @@
+## Reconocimientos
 En adición a los contribuidores de Boost y OpenSSL, y a todos los programadores desde los tiempos inmemoriales, que han hecho este proyecto posible, queremos reconocer lo siguiente:


I had already made some suggestions to this document, but I will suggest them again so more contributors can give their comments or validate them. We have already spoken about not literally translating words because it can change the meaning and not give a coherent structure and affect the meaning of the sentence.

"Adicional a los contribuyentes de Boost y OpenSSL y a todos los programadores que han hecho este proyecto posible desde sus inicios, queremos hacer reconocimiento de lo siguiente:"

lh1008 · 2018-07-30T16:27:20Z

i18n/es/acknowledgements.md

@@ -1,3 +1,4 @@
+## Reconocimientos
 En adición a los contribuidores de Boost y OpenSSL, y a todos los programadores desde los tiempos inmemoriales, que han hecho este proyecto posible, queremos reconocer lo siguiente:

 - Wei Dai, Jeffrey Walton, y todos los desarrolladores en Cryoto++ por proveer una influencial y libre librería de criptografía en C++


"Crypto++" esta mal escrito. En español no utilizamos ",(comas)" precedidos a conjunciones, o solo en casos especiales dado la estructura de lo que queremos decir.

"A Wei Dai, Jeffrey Walton y a todos los desarrolladores de Crypto++ por entregarnos una libreria criptográfica de C++ libre de gran influencia (o, "con una gran influencia")."

Esta es otra manera de escribirlo y darle una mejor estructura gramatical. "Libre libreria", no se lee bien, incluso si buscamos en buscadores el término "libre libreria" no aparece en ninguna parte bajo esa estructura.

Dean Michael Berris, Glyn Matthews, y todos los desarrolladores en cpp-netlib por proveer una librería activamente desarrollada cross-plataforma de red.
" A Dean Michael Berri, Glyn Matthews y a todos los desarrolladoes en cpp-netlib por proporcionar una librería de redes multiplataforma activa".
Ya estamos haciendo enfasis en que unos desarrolladores hicieron una libreria de redes, entonces decir "unos desarrolladores desarrollaron un libreria", suena redundante.

zzz, str4d, y todos los desarrolladores de I2P Java, en ambos, pasado y presente, que tienen un set de estándares a seguir para las demás implementaciones de I2P
"A zzz, str4d y a todos los desarrolladores de Java, que en el presente y en el pasado, establecieron el estándar de todas las otras implementaciones a seguir para I2P".
La estructura de la traducción que se realizo en esta oración pierde el sentido, ya que ellos no "tienen un set de estándares", sino, que ellos "establecieron un estándar", para las implementaciones. Se puede mejorar la estructura incluso de la que sugieron pero si recomiendo mejorar la estructura de la oración original del ingles.
"...who have set the standard for all other I2P implementations to follow" (apartado original del texto en ingles).

orion por proveer i2pcpp: la implementación en C++ original de I2P
"A orion por proveer i2pcpp: la implementación en C++ original de I2P".
He notado que cuando se hacen los reconocimientos no se le agrega la letra "A" al inicio de la oración como formalidad sobre a quien se le otorga el reconocimiento. ejemplo, reconocimientos:

"A" @Josexv1 por haber hecho la traducción.

"A" lh1008 por haber realizado la revisión.

"A" @BlackLotus64 por el interes y las traducciones en la página.
Es una formalidad e indicación de quien escribe hacia alguien o incluso a algo o cosas.
Ejemplo:
Reconocimiento:

"Al" software libre por entregarnos la libertad de pensamiento.

"Al" computador por ser la herramienta que nos permite comunicarnos.

etc.

orignal por proveer i2pd: la mas completa implementación de I2P de la cual poder copiar
"A orignal por proveer i2pd: la mas completa implementación de I2P de la cual poder copiar".

EinMByte por mejorar todas las implementaciones en C++ de I2P (Incluyendo Kovri)
"A EinMByte por mejorar todas las implementaciones en C++ de I2P (Incluyendo Kovri)"

He terminado de agregar las sugerencias, espero las demás correcciones si las hay, para hacer el commit.

Podrías hacer el commint haciendo referencia a los cambios que realizaste en el archivo i18n/es/acknowledgements.md. Asi vamos organizando por partes la revisión. Me parece mas organizado.

Me puedes hacer un ping cuando lo hagas y sigo con la revisión. Asi vamos adelantando de una manera organizada. :) 👍

Josexv1 · 2018-07-30T19:50:18Z

Commit updated @lh1008

lh1008 · 2018-07-31T15:45:13Z

Thank you @Josexv1. I will be doing the review by files. Maybe 3 or 5 files and then finish the review so you can read the comments and wait for other contributors comments, and if you have anything to say we can discuss it. :)

@BlackLotus64 can you help us?

lh1008

Hola @Josexv1,

I was about to continue with the review and it seems the section contributing.md was already translated and you took that version and added it up to the file. I tooked a look at the file developer_guide.md, it was actually updated 6 days ago, and the lines are different (the translation has some missing lines). Could you please take a look and tell me if what I'm saying is correct or am I missing something? :)

Josexv1 · 2018-08-01T18:54:14Z

@lh1008 Tienes toda la razon, al parecer borre algunas lineas, yo el mismo dia que publique el pull-request hice un pull del repo principal. Entonces Fui yo mismo que borre los cambios. En unos minutos lo actualizo.

lh1008 · 2018-08-01T21:54:05Z

Gracias @Josexv1. Quedo atento :).

erciccione · 2018-08-02T19:17:14Z

@lh1008 @Josexv1 Guys please keep it in english, so that everybody (especially non spanish speakers reviewers) can understand.

What's the situation here. Is this ready for a final review?

Josexv1 · 2018-08-02T19:38:06Z

@erciccione i updated the files again to delete some lines missing, and a whole block i left in English. (🥇), now should be ready. @lh1008 can keep checking the rest of the files.

lh1008

Hey @Josexv1,

I think is better if you remove the contributing.md file. That file was replaced by the file developer_guide.md. Did some suggestions. I'll continue tomorrow. :).

@erciccione I think I can finish the review tomorrow. Today I had to do several things so I was not able to work on it very well.

lh1008 · 2018-08-02T20:11:38Z

i18n/es/contributing.md

@@ -1,152 +0,0 @@
-## Seguros de calidad


The file i18n/es/contributing.md should be removed from the translation list files. Is the same as i18n/es/developer_guide.md with missing lines.

lh1008 · 2018-08-02T20:29:59Z

i18n/es/developer_guide.md

-2. For files containing only new work, run [clang-format](http://clang.llvm.org/docs/ClangFormat.html) with ```-style=file``` (which uses our provided [.clang-format](https://github.com/monero-project/kovri/blob/master/.clang-format))
+# Pautas
+- Nuestro objetivo es el cumplimiento completo de C++ 11/14; por favor use esto para su ventaja
+- Utilice la biblioteca estándar y las bibliotecas de dependencia siempre que sea posible


These are two other suggestion that could also work (a recommendation for future translations is not to write two exact words in one sentence). In Spanish two exact words in the same sentence will always mark redundance and affects the grammatical structure. In English redundancy is normal, not in Spanish.

"Utilice la biblioteca estándar y sus dependencias siempre que le sea posible", (le is a pronoun "pronombre refering as an inderect complement to the verb "utilice"; is not a grammatical binding, this is just a suggestion).

or

"Siempre que le sea posible, utilice las librerias estándar y sus dependencias".

lh1008 · 2018-08-02T20:38:03Z

i18n/es/developer_guide.md

 ```bash
-$ cd kovri/ && clang-format -i -style=file src/path/to/my/file
+$ cd kovri / && clang-format -i -style=file /ruta/a/mi/archivo


Remove space after kovri/.

lh1008 · 2018-08-02T20:43:42Z

i18n/es/developer_guide.md

-3. For files with mixed (existing + new) work, run [clang-format](http://clang.llvm.org/docs/ClangFormat.html) selectively over only lines directly related to the new work.
-   - See [vim](http://clang.llvm.org/docs/ClangFormat.html#vim-integration) and [emacs](http://clang.llvm.org/docs/ClangFormat.html#emacs-integration) documentation for examples of configuring keybindings for `clang-format` plugins.
-4. Run [cpplint](https://github.com/google/styleguide/tree/gh-pages/cpplint) (which uses our provided [CPPLINT.cfg](https://github.com/monero-project/kovri/blob/master/CPPLINT.cfg)) to catch any issues that were missed by clang-format
+3. Para archivos con trabajo mixto (existente + nuevo), ejecute [clang-format](http://clang.llvm.org/docs/ClangFormat.html) selectivamente solo sobre las líneas directamente relacionadas con el nuevo trabajo.


Suggestion:
"...seleccionando directamente solo sobre las lineas relacionadas con el nuevo trabajo."

lh1008 · 2018-08-02T21:10:09Z

i18n/es/developer_guide.md

-   - See [vim](http://clang.llvm.org/docs/ClangFormat.html#vim-integration) and [emacs](http://clang.llvm.org/docs/ClangFormat.html#emacs-integration) documentation for examples of configuring keybindings for `clang-format` plugins.
-4. Run [cpplint](https://github.com/google/styleguide/tree/gh-pages/cpplint) (which uses our provided [CPPLINT.cfg](https://github.com/monero-project/kovri/blob/master/CPPLINT.cfg)) to catch any issues that were missed by clang-format
+3. Para archivos con trabajo mixto (existente + nuevo), ejecute [clang-format](http://clang.llvm.org/docs/ClangFormat.html) selectivamente solo sobre las líneas directamente relacionadas con el nuevo trabajo.
+   - Ver la documentacion de [vim](http://clang.llvm.org/docs/ClangFormat.html#vim-integration) y [emacs](http://clang.llvm.org/docs/ClangFormat.html#emacs-integration ) para ejemplos de configuración de combinaciones de teclas para complementos `clang-format`.


Not sure if it's a good idea to translate "plugin". Below you didn't translate it so I think is better to keep consistency, and is hard to translate, it looks like the word "Internet". Maybe a better word will be "extensión" if we decide to translate it.

"...para ejemplos de configuración de combinaciones de teclas para los plugins de clang-format."

"...para ejemplos de configuración de combinaciones de teclas para las extensiones de `clang-format." (still not sure of this one).

lh1008 · 2018-08-02T21:34:15Z

i18n/es/developer_guide.md

  - [cpplint.vim](https://github.com/vim-syntastic/syntastic/blob/master/syntax_checkers/cpp/cpplint.vim)
- Emacs integration
+- Integracion a Emacs


"Integración"

lh1008 · 2018-08-02T21:49:50Z

i18n/es/developer_guide.md

- Use Doxygen three-slash ```/// C++ comments``` when documenting for Doxygen
- Try to document all your work for Doxygen as you progress
- If anonymity is a concern, try to blend in with a present contributor's style
+- Evite el caso mixto ```k``` y el MACRO_TYPE antes de la mezcla para todas las constantes


Suggestions:

"Evite casos mixtos antepuestos k y MACRO_TYPE para todas las constantes" (I go for this one)

or

"Evite, para todas las constantes, lo antepuesto en casos mixtos k y MACRO_TYPE."

lh1008 · 2018-08-02T21:56:07Z

i18n/es/developer_guide.md

-   - If bash programming, read [Google's Shell Style Guide](https://github.com/google/styleguide/blob/gh-pages/shell.xml)
-2. For files containing only new work, run [clang-format](http://clang.llvm.org/docs/ClangFormat.html) with ```-style=file``` (which uses our provided [.clang-format](https://github.com/monero-project/kovri/blob/master/.clang-format))
+# Pautas
+- Nuestro objetivo es el cumplimiento completo de C++ 11/14; por favor use esto para su ventaja


Remove space after "C++11...".

Words as "please" = "por favor" were recommended to be removed in Spanish translations.

"...C++11/14; use esto para su beneficio (recommended)."

lh1008 · 2018-08-02T21:57:58Z

i18n/es/developer_guide.md

- Try to document all your work for Doxygen as you progress
- If anonymity is a concern, try to blend in with a present contributor's style
+- Evite el caso mixto ```k``` y el MACRO_TYPE antes de la mezcla para todas las constantes
+- Utilice Doxygen three-slash ```/// C ++ comments``` al documentar para Doxygen


Remove space after "C++".

"/// C++ comments"

lh1008 · 2018-08-02T22:05:36Z

i18n/es/developer_guide.md

-## TODO's
- Do a quick search in the codebase for ```TODO(unassigned):``` and/or pick a ticket and start patching!
- If you create a TODO, assign it to yourself or write in ```TODO(unassigned):```
+    para la dependencia de los componentes, el aislamiento físico e incluir controles.


Not sure of this.

"checks" means "validation; verification"?

If it's the case I suggest the following:

"... e incluir validaciones."

lh1008

Hi contributors,

This is taking a little more than expected. In the comments, you can read the suggested changes and grammatical corrections needed, with arguments explaining the why on each comment. This is to enforce the translation. Some are minor changes like adding a tilde (accent mark) and others, like taking out repeated words in a sentence that mark redundance, are given to have a better structure in the translated text.
I want to thank @Josexv1 for his efforts and the great job that he has done so far. I will continue later.
If anyone has comments please add them so we can start a discussion.
:)

lh1008 · 2018-08-03T15:02:47Z

i18n/es/developer_guide.md

+   - Actualmente no tenemos ningún tag por estar en Alpha. Por ahora, puedes basar tu trabajo desde la rama master.
+4. Haz cambios
+   - En general, los commits deben ser [atomicos](https://en.wikipedia.org/wiki/Atomic_commit#Atomic_commit_convention) y los diffs deben ser fáciles de leer.
+   - Intente no mezclar correcciones de formato con confirmaciones sin formato


Not sure if it's a good idea to translate the word "commit".

In line 58 you didn't translate the word commit. That is why is better not to translate the word commit.

lh1008 · 2018-08-03T15:06:41Z

i18n/es/developer_guide.md

+   - En general, los commits deben ser [atomicos](https://en.wikipedia.org/wiki/Atomic_commit#Atomic_commit_convention) y los diffs deben ser fáciles de leer.
+   - Intente no mezclar correcciones de formato con confirmaciones sin formato
+5. Sé cortés con el registro de git
+   - El título de commit debe preponer class u otro aspecto del proyecto. Por ejemplo, "HTTPProxy: implementado depurador de User-Agent. Arregla #193." o "Garlic: arregla padding no inicializado en ElGamalBlock".


"class" = "clase"?

lh1008 · 2018-08-03T16:03:41Z

i18n/es/developer_guide.md

+   - Intente no mezclar correcciones de formato con confirmaciones sin formato
+5. Sé cortés con el registro de git
+   - El título de commit debe preponer class u otro aspecto del proyecto. Por ejemplo, "HTTPProxy: implementado depurador de User-Agent. Arregla #193." o "Garlic: arregla padding no inicializado en ElGamalBlock".
+   - Los mensajes del Commit deben ser verbose por default, considerando una línea de asunto de (50 caracteres máximo), una línea en blanco y una explicación detallada como párrafo(s) separado(s) - a menos que el titulo se auto-explique.


The word "verbose" means to express or use more words. The word "verbose" is not a common word so it will sure be hard to understand the meaning if we leave it as it is. Its literal translation "verboso" won't give a description of what it means, so is better to translate it in a phrase.

Phrase suggestion (you might have a better one):
"Los mensajes del commit por defecto deben acompañarse con palabras o una frase descriptiva,..."

The word "considerar/ considerando" means "consider/ considering". "Consisting" means "consiste"; "que consiste de...", "se compone de..." (synonymous).

I believe the word "as" (e.i. the way it should be done) is used as a conjunction in the text, if this is the case it refers, "to indicate by comparison the way that something happens or is done", "...explanatory text as separate paragraph(s)". I suggest to change the word "como" to the word "por" which indicates the "way it should be done". "...una explicación detallada por párrafo(s) detallado(s) ...".

The word "título" has tilde in the "í".

lh1008 · 2018-08-03T16:36:19Z

i18n/es/developer_guide.md

+5. Sé cortés con el registro de git
+   - El título de commit debe preponer class u otro aspecto del proyecto. Por ejemplo, "HTTPProxy: implementado depurador de User-Agent. Arregla #193." o "Garlic: arregla padding no inicializado en ElGamalBlock".
+   - Los mensajes del Commit deben ser verbose por default, considerando una línea de asunto de (50 caracteres máximo), una línea en blanco y una explicación detallada como párrafo(s) separado(s) - a menos que el titulo se auto-explique.
+   - Si un commit en particular hace referencia otro issue, por favor pon una referencia. Por ejemplo, "Leer #123", o "Arregla #123". Esto ayudara a resolver issues cuando se mezclen con ```master```.


In Spanish we recommended to remove the words "please" inside a sentence.

The examples, See #123, or Fixes #123, are inside italic without "apostrophe". Here is the Basic writing and formatting syntax if you would like to take a look at it
Just to leave the comment "See" is "Ver".

The word "ayudará" has tilde in the last "á".

"Si un commit en particular hace referencia a otro issue, pon la/una referencia. Por ejemplo, Leer #123 o Correcciones #123

lh1008 · 2018-08-03T18:59:56Z

i18n/es/developer_guide.md

+   - Los mensajes del Commit deben ser verbose por default, considerando una línea de asunto de (50 caracteres máximo), una línea en blanco y una explicación detallada como párrafo(s) separado(s) - a menos que el titulo se auto-explique.
+   - Si un commit en particular hace referencia otro issue, por favor pon una referencia. Por ejemplo, "Leer #123", o "Arregla #123". Esto ayudara a resolver issues cuando se mezclen con ```master```.
+   - Si un commit en particular se vuelve a establecer después de la colaboración dentro de un pull request, consulte el número de pull request dentro del mensaje del commit. Por ejemplo; *Referencia #123*
+6. [**Firma**](https://git-scm.com/book/en/v2/Git-Tools-Signing-Your-Work) tu(s) commit(s), y si eres un colaborador nuevo, haz un pull request para agregar tu clave PGP a nuestro repositorio.


It's missing "(see contrib)".

lh1008 · 2018-08-03T22:09:56Z

i18n/es/developer_guide.md

+- Un "Colaborador" es una persona que desea proporcionar un parche, siendo un conjunto de compromisos que resuelven algún problema claramente identificado.
+- Un "Mantenedor" es una persona que fusiona parches al proyecto. Los mantenedores no son desarrolladores; Su trabajo es hacer cumplir el proceso.
+- Los colaboradores NO TENDRÁN acceso de commit al repositorio a menos que sean también Mantenedores.
+- Los mantenedores TENDRÁN tener acceso de commit al repositorio.


"...TENDRÁN tener..." is redundant. You can remove the word "tener" and will give the message as it is meant.

Here the same suggestion from above:

"Los mantenderoes TENDRÁN acceso commit del repositorio."

lh1008 · 2018-08-03T22:11:32Z

i18n/es/developer_guide.md

+
+- El proyecto USARÁ el sistema de control de revisión distribuido git.
+- El proyecto ESTARÁ alojado en github.com o equivalente, llamado "Plataforma".
+- El proyecto USARÁ utilizar la plataforma de Tracker de issues.


"...USARÁ utilizar..." is redundant. You can remove the word "utilizar" and will give the message as it is meant. It looks as if this translation was done by a "machine" translator (I don't know how you call those things :)).

lh1008 · 2018-08-03T22:14:14Z

i18n/es/developer_guide.md

+- El proyecto ESTARÁ alojado en github.com o equivalente, llamado "Plataforma".
+- El proyecto USARÁ utilizar la plataforma de Tracker de issues.
+- El proyecto DEBERÁ tener directrices claramente documentadas para el estilo de código.
+- Un "Colaborador" es una persona que desea proporcionar un parche, siendo un conjunto de compromisos que resuelven algún problema claramente identificado.


I suggest to try a bit harder in this translation. By definition in the text, patch is a set of commits that solve a problem.

lh1008 · 2018-08-03T22:16:06Z

i18n/es/developer_guide.md

+- El proyecto USARÁ utilizar la plataforma de Tracker de issues.
+- El proyecto DEBERÁ tener directrices claramente documentadas para el estilo de código.
+- Un "Colaborador" es una persona que desea proporcionar un parche, siendo un conjunto de compromisos que resuelven algún problema claramente identificado.
+- Un "Mantenedor" es una persona que fusiona parches al proyecto. Los mantenedores no son desarrolladores; Su trabajo es hacer cumplir el proceso.


"...; su trabajo..." "s" lowercase.

lh1008 · 2018-08-03T22:55:42Z

i18n/es/developer_guide.md

+- El proyecto DEBERÁ tener directrices claramente documentadas para el estilo de código.
+- Un "Colaborador" es una persona que desea proporcionar un parche, siendo un conjunto de compromisos que resuelven algún problema claramente identificado.
+- Un "Mantenedor" es una persona que fusiona parches al proyecto. Los mantenedores no son desarrolladores; Su trabajo es hacer cumplir el proceso.
+- Los colaboradores NO TENDRÁN acceso de commit al repositorio a menos que sean también Mantenedores.


I suggest:

"Los colaboradores NO TENDRÁN acceso commit del repositorio a menos que sean Mantenedores."

Josexv1 · 2018-08-04T00:57:15Z

It's OK to take time to review it, we don't need to rush things if they're not good enough. I will try to fix all the things tomorrow.

lh1008 · 2018-08-05T19:00:01Z

i18n/es/developer_guide.md

+- El proyecto DEBE utilizar una licencia compartida, como la GPLv3 o una variante de la misma (LGPL, AGPL), o la MPLv2.
+- Todas las contribuciones al código fuente del proyecto ("parches") DEBERÁ utilizar la misma licencia que el proyecto.
+- Todos los parches son propiedad de sus autores. No habrá ningún proceso de asignación de derechos de autor.
+- Los derechos de autor del proyecto serán de propiedad colectiva de todos sus colaboradores.


I found this guide on how to address the names of the people who intervene in a formal contract. They should be addressed with a capital letter.

e.g. Contributor = Contribuyente, Colaborador, Contribuidor; Maintainer = Mantenedor.

lh1008 · 2018-08-05T19:08:04Z

i18n/es/developer_guide.md

+### Licencia y propiedad
+
+- El proyecto DEBE utilizar una licencia compartida, como la GPLv3 o una variante de la misma (LGPL, AGPL), o la MPLv2.
+- Todas las contribuciones al código fuente del proyecto ("parches") DEBERÁ utilizar la misma licencia que el proyecto.


"DEBERÁN" - Found this answer to comment about this actual change.

lh1008 · 2018-08-05T19:17:19Z

i18n/es/developer_guide.md

+
+### Requisitos del parche
+
+- Mantenedores y contribuyentes DEBEN tener una cuenta de la plataforma y DEBERÍAN utilizar sus nombres verdaderos o un alias conocido.


Suggestion:
"Los Mantenedores y los Contribuyentes DEBEN tener una cuenta en la plataforma y DEBERÁN usar sus nombres verdaderos o un alias conocido."

I noticed in this line that the word "contributor" has the translated word "contribuyente" (I have personally supported this translated word), and in the majority of the translated text, the word "colaborador = colaborator" is being used.

- Bump spanish i18n to the latest version - delete some redundant files to match with english folder - fix little typo - Fix wording in acknowledgements - fix developer guides typos

Josexv1 · 2018-08-19T22:19:47Z

Finally found time to update the PR. I added all the suggestions as i don't actually have enough time to check other arguments.

Josexv1 · 2018-09-05T22:21:03Z

Any update on this. I already sent the changes. @erciccione @anonimal

lh1008 · 2018-09-05T22:51:25Z

@Josexv1 I will be starting the review soon (tomorrow I guess). I just finished the user_guides review. Sorry for the delay. But if this needs to be pushed fast, it could go through and we could work changes on the run.

apertamono · 2018-09-06T12:36:07Z

i18n/es/acknowledgements.md

- orion por proveer ```i2pcpp```: la implementación en C++ [original](http://git.repo.i2p.xyz/w/i2pcpp.git) de I2P
- orignal por proveer ```i2pd```: la mas completa implementación de I2P [de la cual poder copiar](https://github.com/purplei2p/i2pd/commit/45d27f8ddc43e220a9eea42de41cb67d5627a7d3)
- EinMByte por mejorar todas las implementaciones en C++ de I2P (Incluyendo Kovri)
+- A Wei Dai, Jeffrey Walton y a todos los desarrolladores de Crypto++ por entregarnos una libreria criptográfica de C++ libre de gran influencia (o, "con una gran influencia").


Please don't leave alternative translations in the final text. I'm not qualified to comment on Spanish grammar, but de gran influencia is a well-known phrase and (o, "con una gran influencia") can probably be simply removed.

apertamono · 2018-09-06T13:41:09Z

All changes suggested by @lh1008 look reasonable to me. And I agree that the current version would be good enough to be merged and improved later.

In developer_guide.md, there are some examples of titles and labels for use on this platform that have been translated while they should be left in English - but those are from the existing contributing.md translation, and I'll make a pull request about this issue for several languages anyway.

Sorry for the late review; I wasn't planning to comment on other languages until I started comparing stuff for my own work in Dutch. I haven't checked everything here, but I have one remark in general: punctuation in bullet points could be more consistent. Either use the same style as the source text or choose your own style, e.g. add a period after each full sentence containing a verb.

anonimal · 2018-09-07T09:04:50Z

NOTICE: THIS PULL REQUEST HAS BEEN MOVED TO GitLab. Please reopen there if you wish to merge. See #102 for details.

anonimal requested changes Jul 28, 2018

View reviewed changes

Josexv1 force-pushed the master branch from 304eb42 to 8595a9d Compare July 28, 2018 23:14

lh1008 suggested changes Jul 30, 2018

View reviewed changes

Josexv1 force-pushed the master branch from 8595a9d to 31fb26e Compare July 30, 2018 19:49

lh1008 reviewed Jul 31, 2018

View reviewed changes

Josexv1 force-pushed the master branch 2 times, most recently from 0e6ac46 to c549b05 Compare August 1, 2018 19:52

lh1008 reviewed Aug 3, 2018

View reviewed changes

lh1008 reviewed Aug 5, 2018

View reviewed changes

Update Spanish i18n

64a5348

- Bump spanish i18n to the latest version - delete some redundant files to match with english folder - fix little typo - Fix wording in acknowledgements - fix developer guides typos

Josexv1 force-pushed the master branch from c549b05 to 64a5348 Compare August 19, 2018 22:18

apertamono reviewed Sep 6, 2018

View reviewed changes

anonimal closed this Sep 7, 2018

		@@ -1,3 +1,4 @@
		## Reconocimientos
		En adición a los contribuidores de Boost y OpenSSL, y a todos los programadores desde los tiempos inmemoriales, que han hecho este proyecto posible, queremos reconocer lo siguiente:


		### Requisitos del parche

		- Mantenedores y contribuyentes DEBEN tener una cuenta de la plataforma y DEBERÍAN utilizar sus nombres verdaderos o un alias conocido.

Update Spanish i18n #88

Update Spanish i18n #88

Conversation

Josexv1 commented Jul 27, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

erciccione commented Jul 29, 2018 • edited Loading

BlackLotus64 commented Jul 29, 2018

lh1008 commented Jul 30, 2018

lh1008 commented Jul 30, 2018

lh1008 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

lh1008 Jul 30, 2018 • edited Loading

Choose a reason for hiding this comment

Josexv1 commented Jul 30, 2018

lh1008 commented Jul 31, 2018

lh1008 left a comment

Choose a reason for hiding this comment

Josexv1 commented Aug 1, 2018

lh1008 commented Aug 1, 2018

erciccione commented Aug 2, 2018

Josexv1 commented Aug 2, 2018

lh1008 left a comment • edited Loading

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

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

lh1008 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

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

Choose a reason for hiding this comment

Josexv1 commented Aug 4, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Josexv1 commented Aug 19, 2018

Josexv1 commented Sep 5, 2018 • edited Loading

lh1008 commented Sep 5, 2018

Choose a reason for hiding this comment

apertamono commented Sep 6, 2018 • edited Loading

anonimal commented Sep 7, 2018

erciccione commented Jul 29, 2018 •

edited

Loading

lh1008 Jul 30, 2018 •

edited

Loading

lh1008 left a comment •

edited

Loading

Josexv1 commented Sep 5, 2018 •

edited

Loading

apertamono commented Sep 6, 2018 •

edited

Loading