Promises chaining by cortizg · Pull Request #189 · javascript-tutorial/es.javascript.info · GitHub
Skip to content

Promises chaining#189

Merged
vplentinax merged 9 commits intojavascript-tutorial:masterfrom
cortizg:es.javascript.info.1-11-03-pc
Jun 10, 2020
Merged

Promises chaining#189
vplentinax merged 9 commits intojavascript-tutorial:masterfrom
cortizg:es.javascript.info.1-11-03-pc

Conversation

@cortizg
Copy link
Copy Markdown
Contributor

@cortizg cortizg commented May 14, 2020

No description provided.

@cortizg
1-11-03-pc Traducido 11
ad176f6 
1-11-03-pc Corregido 8

1-11-03-pc Corregido 8

coregido typos
@javascript-translate-bot javascript-translate-bot requested a review from a team May 14, 2020 17:00
@iliakan iliakan mentioned this pull request May 14, 2020
Open
cortizg
cortizg commented Jun 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor Author

@cortizg cortizg left a comment

Choose a reason for hiding this comment

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

cambio sugerido

Comment thread 1-js/11-async/03-promise-chaining/01-then-vs-catch/solution.md Outdated
cortizg
cortizg commented Jun 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor Author

@cortizg cortizg left a comment

Choose a reason for hiding this comment

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

es un valor cadena de un symbol, se puede traducir pero prefiero dejarlo a como estaba

Comment thread 1-js/11-async/03-promise-chaining/user.json Outdated
cortizg
cortizg commented Jun 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor Author

@cortizg cortizg left a comment

Choose a reason for hiding this comment

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

detalles y typos

Comment thread 1-js/11-async/03-promise-chaining/article.md Outdated
Comment thread 1-js/11-async/03-promise-chaining/article.md Outdated
Comment thread 1-js/11-async/03-promise-chaining/article.md Outdated
Comment thread 1-js/11-async/03-promise-chaining/article.md Outdated
Comment thread 1-js/11-async/03-promise-chaining/article.md Outdated
joaquinelio
joaquinelio approved these changes Jun 10, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

Muy bueno.
algunas opiniones, no insisto
Algunos conflictos fácil de resolver, apruebo en advance aunque no deba, es para que o molesten más.

Comment thread 1-js/11-async/03-promise-chaining/article.md
Comment thread 1-js/11-async/03-promise-chaining/article.md
Copy link
Copy Markdown
Member

@joaquinelio joaquinelio Jun 10, 2020

Choose a reason for hiding this comment

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

"...¿especialización?"
"Soy frontend... eh... programador de interfaces"

Copy link
Copy Markdown
Contributor Author

@cortizg cortizg Jun 10, 2020

Choose a reason for hiding this comment

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

Podría poner en el "ambiento de desarrollo"..... Entorno de programación.... Me gusta más.... El problema es que confíaba demasiado en la traducciones google, confíaba😱

Copy link
Copy Markdown
Contributor Author

@cortizg cortizg Jun 10, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

Además JS está en el frontend y en el backend con node y creo que no se usan promesas en node; está bien el contexto porque lo resaltan....node en su núcleo no procesa promesas

Copy link
Copy Markdown
Contributor Author

@cortizg cortizg Jun 11, 2020

Choose a reason for hiding this comment

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

tambien podria dejarse como "programacion en el frontend", no se, comentarios?

@vplentinax vplentinax merged commit bc831f3 into javascript-tutorial:master Jun 10, 2020
@javascript-translate-bot
Copy link
Copy Markdown

javascript-translate-bot commented Jun 10, 2020

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

Reviewers

@joaquinelio joaquinelio joaquinelio approved these changes

Assignees

No one assigned

Labels

needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cortizg @javascript-translate-bot @joaquinelio @vplentinax