Skip to content

ReactDOM page translation - #48

Merged
fjoshuajr merged 5 commits into
reactjs:masterfrom
michellocana:master
Feb 11, 2019
Merged

ReactDOM page translation#48
fjoshuajr merged 5 commits into
reactjs:masterfrom
michellocana:master

Conversation

@michellocana

@michellocanamichellocana commented Feb 9, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@fjoshuajrfjoshuajr added the needs review A pull request ready to be reviewed label Feb 9, 2019
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/reference-react-dom.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/reference-react-dom.md Outdated
glaucia86
glaucia86 suggested changes Feb 10, 2019
View reviewed changes

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

@michellocana

Nós não suportamos navegadores antigos que não suportam métodos ES5, mas você pode descobrir que suas aplicações funcionam em navegadores antigos se polyfills como es5-shim and es5-sham forem incluídos na página. Você está por sua conta se optar por seguir esse caminho.

Comment threadcontent/docs/reference-react-dom.md Outdated
@glaucia86glaucia86 added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Feb 10, 2019
@glaucia86

glaucia86 commented Feb 10, 2019

Copy link
Copy Markdown
Contributor

@michellocana

Algumas alterações a serem feitas. Pedimos que faça os ajustes e logo iremos aprovar a sua contribuição.
Pedimos que, conforme for realizando os acertos que marque como 'Resolve Conversation' e nos sinalize para que possamos analisar novamente a tradução e enfim, aprovar o seu PR. Muito obrigada!

Assim que você concluir as suas mudanças, marque como 'Resolve Conversation' e altere para a label: 'changes already resolved it - translator'.
Muito obrigada pela sua contribuição! 😄

rafael-vitorand others added 2 commits February 10, 2019 09:41
@rafael-vitor@michellocana
Fixing translation
cd84df8
Co-Authored-By: michellocana <ocana.1997@hotmail.com>
@rafael-vitor@michellocana
Fixing hydrate section translation
f76441c
Co-Authored-By: michellocana <ocana.1997@hotmail.com>
@tesseralistesseralis mentioned this pull request Feb 10, 2019
Closed
90 tasks
@michellocana

michellocana commented Feb 10, 2019

Copy link
Copy Markdown
ContributorAuthor

@rafael-vitor@glaucia86 obrigado pelas sugestões, fiz os ajustes necessários 😁

@glaucia86 fiz um pequeno ajuste na sua sugestão, na expressão "por conta e risco"

@WendellAdrielWendellAdriel added feedback addressed and removed needs author response Changes requested needing author's input labels Feb 11, 2019
glaucia86
glaucia86 previously approved these changes Feb 11, 2019
View reviewed changes

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

LGTM 👍

@glaucia86glaucia86 added needs +1 Needs another approval before merging and removed feedback addressed labels Feb 11, 2019
fjoshuajr
fjoshuajr approved these changes Feb 11, 2019
View reviewed changes

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

LGTM ✨✨

@fjoshuajrfjoshuajr added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Feb 11, 2019
@fjoshuajr

fjoshuajr commented Feb 11, 2019

Copy link
Copy Markdown
Contributor

LGTM 👍

Vou fazer merge ✌

@fjoshuajr
fjoshuajr merged commit 14ce6fe into reactjs:masterFeb 11, 2019
@michellocanamichellocana changed the title ReactDOM page translationWIP: ReactDOM page translationFeb 11, 2019
@michellocanamichellocana changed the title WIP: ReactDOM page translationReactDOM page translationFeb 11, 2019
@michellocana

michellocana commented Feb 11, 2019

Copy link
Copy Markdown
ContributorAuthor

@fjoshuajr acabei de ver um "component" na mesma frase que tu ajustou, mas agora que foi dado merge, acho que posso fazer uma PR separada como fix

@fjoshuajr

fjoshuajr commented Feb 11, 2019•
edited
Loading

Copy link
Copy Markdown
Contributor

Sim @michellocana, pode fazer um PR.

Vou aprovar.

devin-ai-integrationBot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@michellocana@fjoshuajr
ReactDOM page translation (reactjs#48)
bb879e8
* ReactDOM page translation
* Fixing translation
Co-Authored-By: michellocana <ocana.1997@hotmail.com>
* Fixing hydrate section translation
Co-Authored-By: michellocana <ocana.1997@hotmail.com>
* Fixing browser support section translation
* Component -> Componente
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers
@rafael-vitorrafael-vitorrafael-vitor left review comments
@fjoshuajrfjoshuajrfjoshuajr approved these changes
@glaucia86glaucia86glaucia86 left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

🌐 translatedTranslation is completed for this page

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@michellocana@glaucia86@fjoshuajr@rafael-vitor@WendellAdriel