Skip to content

Scripts: async, defer - #324

Merged
joaquinelio merged 26 commits into
javascript-tutorial:masterfrom
dbritto-dev:master
Jul 29, 2020
Merged

Scripts: async, defer#324
joaquinelio merged 26 commits into
javascript-tutorial:masterfrom
dbritto-dev:master

Conversation

@dbritto-dev

Copy link
Copy Markdown
Contributor

No description provided.

@CLAassistant

CLAassistant commented Jul 26, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@iliakaniliakan mentioned this pull request Jul 26, 2020
@joaquinelio
joaquinelio self-requested a review July 27, 2020 03:47

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

Está bien, typos y tildes

podes aceptar, pelearme o ignorarme...
pero...
Creo que te ofreciste para review?
de los PR, con etiquetas "review needed", son los que no tuvieron revisión
o la tuvieron y el autor hizo/rechazó los cambios
Si está esperando "change req" no creo que valga la pena revisar, que el autor revise primero

Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
Comment thread2-ui/5-loading/02-script-async-defer/article.md Outdated
@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 👻

dbritto-devand others added 19 commits July 27, 2020 00:02
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>
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>
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>
dbritto-devand others added 3 commits July 27, 2020 00:48
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@dbritto-dev

Copy link
Copy Markdown
ContributorAuthor

/done

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

Ok.
autor y/o 2do reviewer,
*clic para arreglar nro linea, algo menor
*No estoy convencido de la tabla en 185 (de origen), pero el evento y la relacion con los scripts se explican bien en el art.
Más que ok.

Comment thread2-ui/5-loading/02-script-async-defer/article.md
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@dbritto-dev

Copy link
Copy Markdown
ContributorAuthor

/done

@vplentinaxvplentinax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok!

@javascript-translate-bot

Copy link
Copy Markdown

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

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.

5 participants

@dbritto-dev@CLAassistant@javascript-translate-bot@joaquinelio@vplentinax