Skip to content

Conversation

@brunodahora
Copy link
Contributor

@brunodahora brunodahora commented Oct 9, 2019

Só algumas observações.

  • Preferi não traduzir profiler.
  • Achei melhor traduzir os nomes dos gráficos.

close #277

@netlify
Copy link

netlify bot commented Oct 9, 2019

Deploy preview for pt-br-reactjs ready!

Built with commit 6ad6c97

https://deploy-preview-336--pt-br-reactjs.netlify.com

@jhonmike jhonmike mentioned this pull request Oct 10, 2019
45 tasks
@jhonmike jhonmike added needs review A pull request ready to be reviewed blog Blog post translation labels Oct 10, 2019
@jhonmike jhonmike added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Oct 17, 2019
myialine
myialine previously approved these changes Oct 24, 2019
Copy link
Contributor

@myialine myialine left a comment

Choose a reason for hiding this comment

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

A tradução está muito boa! Deixei umas sugestões para ficar ainda melhor.

> Read more about how to use this bundle at [fb.me/react-profiling](https://fb.me/react-profiling)
> `react-dom` 16.5+ suporta análise no modo DEV.
> Um bundle de produção para análise está disponível como `react-dom/profiling`.
> Leia mais sobre como usar este bundle em [fb.me/react-profiling](https://fb.me/react-profiling)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> Leia mais sobre como usar este bundle em [fb.me/react-profiling](https://fb.me/react-profiling)
> Leia mais sobre como usar este pacote em [fb.me/react-profiling](https://fb.me/react-profiling)

@jhonmike
Copy link
Member

@brunodahora chegou a verificar as sugestões propostas no seu PR? pois o PR esta praticamente pronto e logo podemos fazer merge, basta verificar as sugestões e aceitar as que achar valida!

@brunodahora
Copy link
Contributor Author

Oi gente, desculpe o sumiço. Vou revisar hj e já retorno para aprovação.

Co-Authored-By: Jhon Mike <[email protected]>
Co-Authored-By: Aline Cavalcante <[email protected]>
@brunodahora
Copy link
Contributor Author

Obrigado pelas revisões gente.
@jhonmike apliquei as sugestões.

Copy link
Member

@jhonmike jhonmike left a comment

Choose a reason for hiding this comment

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

lgtm

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Feb 4, 2020
@jhonmike jhonmike merged commit beca261 into reactjs:master Feb 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blog Blog post translation 🌐 translated Translation is completed for this page

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Translate 2018-09-10-introducing-the-react-profiler.md

3 participants