Form properties and methods

[kenliten]

Traducido Form elements

[joaquinelio]

todavia no lo revise, pero aviso que tiene task y solution para traducir tambien
y por las dudas: los svg no se tocan, tienen tratamiento especial

[joaquinelio]

El nombre para que Don Bot lo registre es "Form properties and methods", puedo agregarlo a mano al issue pero

Faltan task y solution, podrias

• agregarlos aca y corrijo nombre e issue a mano, o
• hacer un nuevo pr con ambos y el nombre "Form properties and methods"
  y hago merge de este sin cambiar el nombre
• No vale la pena borrar este... que ya tiene la revision

[javascript-translate-bot]

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

[joaquinelio]

MERGE:
a quien lo haga, plis verificar formatos anidados online
imposible de verificar dentro del hub

[kenliten]

El nombre para que Don Bot lo registre es "Form properties and methods", puedo agregarlo a mano al issue pero

Faltan task y solution, podrias

* agregarlos aca y corrijo nombre e issue a mano, o

* hacer un nuevo pr con ambos y el nombre "Form properties and methods"
  y hago merge de este sin cambiar el nombre

* No vale la pena borrar este... que ya tiene la revision

No entiendo a que te refieres con "Task y Solution" me aclaras?

[joaquinelio]

Puedo.
los articulos tienen un subdirectorio por cada ejercicio. El tuyo tiene
article.md que ya hiciste,
un svg que no se toca,
y /1-add-select-option/ con task.md y solution.md dentro

https://github.com/kenliten/es.javascript.info/blob/kenliten/2-ui/4-forms-controls/1-form-elements/1-add-select-option/task.md
https://github.com/kenliten/es.javascript.info/blob/kenliten/2-ui/4-forms-controls/1-form-elements/1-add-select-option/solution.md

ah, el encabezado "importance: x" del task no se traduce aunque no se para que sirve.
happy translating 🎉

[kenliten]

Puedo.
los articulos tienen un subdirectorio por cada ejercicio. El tuyo tiene
article.md que ya hiciste,
un svg que no se toca,
y /1-add-select-option/ con task.md y solution.md dentro

https://github.com/kenliten/es.javascript.info/blob/kenliten/2-ui/4-forms-controls/1-form-elements/1-add-select-option/task.md
https://github.com/kenliten/es.javascript.info/blob/kenliten/2-ui/4-forms-controls/1-form-elements/1-add-select-option/solution.md

ah, el encabezado "importance: x" del task no se traduce aunque no se para que sirve.
happy translating tada

Ok, perfecto.

Entonces deja los traduzco, y asi hago el pr con el ultimo que tome que ya lo tengo listo y estos tres en un solo pr.
💯 PeAcE!

[joaquinelio]

@kenliten
hello... anybody home?
me gustaria agregar esto, pero faltan task y solution

o @vplentinax podes revisarlo,
hacemos merge y agrego lo faltante como issue, ¡es muy poco lo que falta! Es una pena no aprovecharlo.

merge: no figura en el issue17, hay que agregarlo a mano

[joaquinelio]

@kenliten oops!
Te había comentado que otro articulo requiere otro branch

he aquí

-----master no se toca, cada tanto haces

git fetch upstream
git merge upstream/master

----para cada articulo un branch

git branch articulo1
git checkout articulo1
----mods,,, stage, commits...
git push origin articulo1
------ pr1

----depues de esto modificar el branch y push es modificar el pr

------- (ojo si quedaron cambios sin commit en el limbo se arrastran con el cambio de branch)
git checkout master
---- actualizas merge si hace falta

git branch articulo2
git checkout articulo2
----mods,,, stage, commits...
git push origin articulo2
------- pr2

El tema es mantener los PR simples y bajo control, tantos archivos en un solo pr se hace dificil de revisar
PR cortos se mueven mas rapido

[kenliten]

De acuerdo, perdón, ando algo messy, jaja, voy de nuevo

[joaquinelio]

De acuerdo, perdón, ando algo messy, jaja, voy de nuevo

¿sabes como hacerlo?
Me paso exactamente lo mismo hace un año #105
me ayudaron y ahi aprendi bastante de git...
recomiendo ver la solucion ayuda a entender los commit
es buen ejercicio practicarlo

pero ya habia solucionado cortando y pegando como un niño de 5,
mas rapido porque sí tenia claro qué estaba haciendo (hay que ser cuidadoso para no perder nada)

como sea, cuidado no rompas este pr que tiene ya mucho trabajo tuyo y de revision

tambien podria revisar lo que falta y ya, no es para tanto. demoramos un poco
pero no se si mi colega reviewer estaria de acuerdo

[joaquinelio]

esta bueno
sobran algunas comas qe salvo la obvia no las corrijo
porque es batalla perdida =(

a ver si agregamos de una vez...

falta tan poco...

[javascript-translate-bot]

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

[joaquinelio]

no me voy a poner exquisito
ya lo revise asi, doble pr

onmerge solo hay que agregar el tilde en el issue 17

[vplentinax]

Bien

[javascript-translate-bot]

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