Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Node properties: type, tag and contents#422

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

homero304
Copy link
Contributor

please review my PR

Copy link
Member

@joaquineliojoaquinelio left a comment

Choose a reason for hiding this comment

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

correcciones y divagues...

y sorry si algun coment parece agresivo no lo es, solo estoy jugando.
es que nunca antes en mi vida usé bugs, era perfecto
y en las ultimas sync puse dos y todavia no me recupero del shock.

Ah, creo que hay correctores automaticos metiendo el rabo
bad Homer..!

@javascript-translate-bot

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

omero329and others added11 commitsOctober 12, 2020 06:31
…ion.mdCo-authored-by: joaquinelio <joaquinelio@gmail.com>
…solution.mdCo-authored-by: joaquinelio <joaquinelio@gmail.com>
…in-hierarchy/solution.mdCo-authored-by: joaquinelio <joaquinelio@gmail.com>
…in-hierarchy/solution.mdCo-authored-by: joaquinelio <joaquinelio@gmail.com>
…in-hierarchy/task.mdCo-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>
omero329and others added7 commitsOctober 12, 2020 06: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>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@homero304
Copy link
ContributorAuthor

/done

@joaquinelio
Copy link
Member

joaquinelio commentedOct 12, 2020
edited
Loading

ah,
lo de "built in" era solo un comentario

"incorporada" es lo que usamos hasta ahora, seria "correcto" aunque en mi opinion muy facil de malinterpretar, las bibliotecas de 3ros tambien "incorporan" funciones
"integrada" me gustaba un poco más, no se por qué, me da idea de que ya esta ahi, no incorporado posteriormente en una biblioteca extra... "built-in" ="integrada" me gusta. No recuerdo ssi ses uso asi en algun lado.
"nativa" me parecia perfecta pero no lo habiamos ussado antes, y no lo discutimos antes, para mi esta bien, no lo habia sugerido porque podria haber desacuerdos,
Acepto cualquier version, pero no se los demás.

winnie pooh
jaj el comment que hice era por la mayuscula, en realidad si el codigo es coherente no me importaba. Sigue ok.
no se por que me pongo a hacer esos comentarios...
sorry

Copy link
Contributor

@vplentinaxvplentinax left a comment

Choose a reason for hiding this comment

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

Good!

@joaquineliojoaquinelio merged commitead3c54 intojavascript-tutorial:masterOct 15, 2020
@javascript-translate-bot

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

@joaquinelio
Copy link
Member

joaquinelio commentedOct 15, 2020
edited
Loading

@vplentinax
🎉🎉🎉🎉🎉
Si sos el 2do ok, podes hacer merge sin esperarme,
salvo que no tengas tiempo de revisar online, ya cometi el error de no revisar bien, dejamelo es una satisfacción.
*El mensaje de idioma sigue saliendo, no problem, al tiempo deja de salir.
*Tambien tarda un poco más mostrar el índice español que la página. El índice ES el titulo de la pagina, asi que solo es esperar nosequé.
*Siempre revisar FORMATO no se ven los err de formato dentro de git
*Imagenes
*Las soluciones, que abran, ves enseguida si falta traduccion, y TAMBIEN he visto fallas de formato dentro
*Los links, suelo buscarlos y abrirloss con el boton del medio asi reviso todos juntos, internos y externos
-Los sandbox... culpable, demora mucho en abrir plunker y a veces son muchos...
*Que salga el msg del bot del issue17, para actualizar a mano si no lo ve por algo.

Y ESTE PR se vio perfecto
¡¡¡¡ Faltan 25 !!!!
🎉🎉🎉🎉🎉

vplentinax reacted with hooray emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@joaquineliojoaquineliojoaquinelio approved these changes

@vplentinaxvplentinaxvplentinax approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@homero304@javascript-translate-bot@joaquinelio@vplentinax@omero329

[8]ページ先頭

©2009-2025 Movatter.jp