All committers have signed the CLA.

Choose a reason for hiding this comment

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

• Varios acentos, si no usas las suggest acordate de ponerlos
• Recordar que faltan task y solution
• Me permití corregir los nros de linea
  necesitamos que coincidan con el inglés para poder hacer bien los pull
• Cambie nombre para que Don Bot lo reconozca, agregué el pr que Don Bot no reconoció al issue17

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

Buenas Joaquin, ahí realicé un par de actualizaciones tomando tus sugerencias.

Cualquier cosa avisame

Uh
Debí advertirte, no traducimos código, puede salir mal.
Y no lo podemos probar acá.
(Pero los devs somos perfectos y no cometemos errores, para qué probar)

Se traducen solamente // comentarios /* */
y usualmente mensajes entrecomillados (con cuidado los backticks) pero no los string que participan de codigo

salvo eso, el article creo que ya está, task y solution estan bien...
¿podrias poner codigo sin traducir?

Choose a reason for hiding this comment

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

Qudó un js
5-network/01-fetch/01-fetch-users/_js.view/solution.js
lo más seguro es poner el original entero,

al otro js le copie la linea vieja,
5-network/01-fetch/01-fetch-users/_js.view/source.js
por lo menos el comentario quedo traducido... =)

Estaba tentado a dejarlo asi, probarlo localmente
el problema es que a veces hacen cambios de codigo y el pull es directo
es posible por ej que agreguen una linea usando la variable ingles y nadie se de cuenta de que el codigo se malogro

Mas par de cositas,

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

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Listo. Perdón por las idas y vueltas.

/done

Choose a reason for hiding this comment

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

@joaquinelio por las dudas, no me deja completar el PR. Supongo que hará falta que otra persona con permisos revise el PR y lo complete. Saludos!

Sí, solo maintainers pueden continuarlo.

• El branch y el PR son tuyos, podes hacer y deshacer.
• merge dentro del repo solo la puede hacer alguien con permisos,
• la aprobación es cuestion de procedimiento, lo esperable es que dos personas aprueben y el bot de Ilya responde cambiando los cartelitos.
• Vos podrias hacer review y aprobar el PR de otra persona, y aunque el BOT te ignore, el maintainter no
  (especialmente por la escasez de reviewers)

Ja, deberia poner esto en el curso de git filosofico
para complementar cualquier curso de comandos git

Choose a reason for hiding this comment

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

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