Skip to content

Translation: Context - #55

Merged
fjoshuajr merged 5 commits into
reactjs:masterfrom
jancassio:master
Feb 12, 2019
Merged

Translation: Context#55
fjoshuajr merged 5 commits into
reactjs:masterfrom
jancassio:master

Conversation

@jancassio

@jancassiojancassio commented Feb 10, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@jancassiojancassio reopened this Feb 10, 2019
@fjoshuajrfjoshuajr added the needs review A pull request ready to be reviewed label Feb 10, 2019
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/context.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/context.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/context.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/context.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes
Comment threadcontent/docs/context.md Outdated
rafael-vitor
rafael-vitor reviewed Feb 10, 2019
View reviewed changes

@rafael-vitorrafael-vitor 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.

@jancassio fiz algumas sugestões. Bom trabalho!

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.

@jancassio

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!

Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.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 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.

@jancassio tudo que tiver 'Context' coloque sempre assim: Contexto (context), okay?

Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
Comment threadcontent/docs/context.md Outdated
@glaucia86

glaucia86 commented Feb 10, 2019

Copy link
Copy Markdown
Contributor

@jancassio

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! 😄

@jancassio

jancassio commented Feb 10, 2019

Copy link
Copy Markdown
ContributorAuthor

@glaucia86 revisão ta feita, só não consigo alterar a label por que não estou como contribuidor deste projeto (eu acho).

@WendellAdrielWendellAdriel added feedback addressed and removed needs author response Changes requested needing author's input labels Feb 11, 2019
glaucia86
glaucia86 approved these changes Feb 12, 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 👍

cc: @cezaraugusto@WendellAdriel@fjoshuajr podem aprovar.

@glaucia86glaucia86 added needs +1 Needs another approval before merging and removed feedback addressed labels Feb 12, 2019
fjoshuajr
fjoshuajr approved these changes Feb 12, 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.

Muito bom! 🎉🎉

@fjoshuajrfjoshuajr added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Feb 12, 2019
@tesseralistesseralis mentioned this pull request Feb 12, 2019
Closed
90 tasks
@fjoshuajr
fjoshuajr merged commit 639c06a into reactjs:masterFeb 12, 2019
@jancassio

jancassio commented Feb 13, 2019

Copy link
Copy Markdown
ContributorAuthor

iiiieeeeyyyyy 🎉🎉🎉
Muito obrigado gente.

devin-ai-integrationBot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@jancassio@fjoshuajr
Translation: Context (reactjs#55)
72e3029
* Translate: Context
* Translate: Context first review
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
@glaucia86glaucia86glaucia86 approved these changes
@fjoshuajrfjoshuajrfjoshuajr approved these changes
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

@jancassio@glaucia86@fjoshuajr@rafael-vitor@WendellAdriel