Conversation
merge from repo
fetch article
|
Hola uno ejercicio solo. en tu branch |
There was a problem hiding this comment.
- 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". |
|
Buenas Joaquin, ahí realicé un par de actualizaciones tomando tus sugerencias. Cualquier cosa avisame |
|
Uh Se traducen solamente // comentarios /* */ salvo eso, el article creo que ya está, task y solution estan bien... |
|
@joaquinelio Creo que ahí estamos. Avisame! |
joaquinelio
left a comment
There was a problem hiding this comment.
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". |
Co-authored-by: joaquinelio <[email protected]>
Co-authored-by: joaquinelio <[email protected]>
|
Listo. Perdón por las idas y vueltas. /done |
|
@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.
Ja, deberia poner esto en el curso de git filosofico |
|
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
No description provided.