FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Destructuring assignment by EzequielCaste · Pull Request #259 · javascript-tutorial/es.javascript.info · GitHub

Destructuring assignment - #259

Merged
EzequielCaste merged 20 commits into
javascript-tutorial:masterfrom
EzequielCaste:destructuring-assig
Jul 8, 2020
Merged

Destructuring assignment#259
EzequielCaste merged 20 commits into
javascript-tutorial:masterfrom
EzequielCaste:destructuring-assig

Conversation

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #17, it's 163 ⁉️

joaquinelio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

falta traducir 1-5-10-1 task (movido hace poco)
faltan traducir líneas de article
Apareció key para mortificarme... pero acá zafa, asi que no comenté.
¿Te parece que corrija chiquiteces como signos asimétricos ¡!, espacios extras, inicios en mayúscula y cosas así? ¿O preferís una revisión más blanda y rápida? Me adapto a ambas, lo único prioritario es la precisión en JS.

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Copy link
Copy Markdown
Contributor Author

falta traducir 1-5-10-1 task (movido hace poco)
faltan traducir líneas de article
Apareció key para mortificarme... pero acá zafa, asi que no comenté.
¿Te parece que corrija chiquiteces como signos asimétricos ¡!, espacios extras, inicios en mayúscula y cosas así? ¿O preferís una revisión más blanda y rápida? Me adapto a ambas, lo único prioritario es la precisión en JS.

y quíen estableció esa prioridad? hubo votación y no me notificaron... que fea actitud.

@joaquinelio repasemos cómo hacer un Review tu trabajo es revisar y corregir errores. Espacios, mayúsculas, números de líneas, gramática, ortografía, etc.
Por suerte tenemos la herramienta de sugerir cambios:

Recomiendo usar esa herramienta siempre.
Lo ideal sería que el autor de la traducción corrija sus errores así puede ver en qué se equivoco y tratar de mejorar sus próximas traducciones.

EzequielCaste and others added 3 commits July 6, 2020 09:15
traducción de task
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
EzequielCaste removed the request for review from vplentinax July 6, 2020 12:41
EzequielCaste and others added 5 commits July 6, 2020 09:42
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
EzequielCaste and others added 2 commits July 6, 2020 09:47
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>

EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

cambios

EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

ok

Copy link
Copy Markdown
Member

Estamos chocando más seguido eso es bueno. Es porque ya limpiamos todo ¡este es el más viejo PR, de 20 días!
Solo salto a uno más nuevo cuando veo uno que no es del trio fantásico, lo que desanimó el año pasado a tantos altruistas es la falta de realimentación. Recomiendo prestar especial atención a los pr de los nuevos.

Prioridad precisión JS. Me expresé mal, es la mia. La votación la resigné en uds.
1 precision JS
2 claridad. Aunque se aparte de EN pero con cuidado, no corresponde reinterpretar.
3 glosario, buen sistema de nombres... bue, no pudo ser
4 literariamente correcto. Prefiero el "good enough" a que se abandone por revisiones exigentes.
Bien puedo saltear un // Bravo!
o una coma entre sujeto y verbo.

REVIEW es un EMBOLE pero hay escasez, asi que me resigno aunque no me queda gana para además hacer nuevas. Lo peor en tiempo y frustración, tildes. Suerte que uds son buenos en eso y también en lo conceptual de JS, pocos arreglos ahí.

La suggest la uso siempre que puedo: cuando sé. Si no, solo pongo la queja 😄

joaquinelio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

ok, creo. O estoy dormido.

vplentinax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

ok!

EzequielCaste merged commit b02ef3a into javascript-tutorial:master Jul 8, 2020

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL