Skip to content

Promise - #188

Closed
cortizg wants to merge 2 commits into
javascript-tutorial:masterfrom
cortizg:es.javascript.info.1-11-02-p
Closed

Promise#188
cortizg wants to merge 2 commits into
javascript-tutorial:masterfrom
cortizg:es.javascript.info.1-11-02-p

Conversation

@cortizg

Copy link
Copy Markdown
Contributor

No description provided.

corregido detalles
corregido typo

@joaquineliojoaquinelio 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.

Repo borrado. no puedo hacer suggest (dejé 3 comments)
@cortizg
Cambios menores, podés ignorarlos y pedir merge...
aunque no creo, son más estrictos que yo

Falta 1/11/2/2 solution, un solo renglón
typo
acento
elegir 1 palabra por consistencia /manejador/controlador/

EDIT: por qué hojarasca sale la letra dl coment tan grande si no puse ninguna decoracion

Comment thread1-js/11-async/02-promise-basics/article.md
Comment thread1-js/11-async/02-promise-basics/article.md
Comment thread1-js/11-async/02-promise-basics/article.md
@javascript-translate-bot

Copy link
Copy Markdown

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

@cortizg

Copy link
Copy Markdown
ContributorAuthor

Repo borrado. no puedo hacer suggest (dejé 3 comments)

@cortizg
Cambios menores, podés ignorarlos y pedir merge...
aunque no creo, son más estrictos que yo

Falta 1/11/2/2 solution, un solo renglón

typo
acento
elegir 1 palabra por consistencia /manejador/controlador/

EDIT: por qué hojarasca sale la letra dl coment tan grande si no puse ninguna decoracion

Tendré que cerrar el PR y subirlo de nuevo el branch

@cortizg

Copy link
Copy Markdown
ContributorAuthor

Cerrado por falta de branch

@cortizgcortizg closed this Jun 29, 2020
@cortizgcortizg mentioned this pull request Jun 29, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@cortizg@javascript-translate-bot@joaquinelio