Skip to content

Conditional Rendering translation#51

Merged
WendellAdriel merged 14 commits into
reactjs:masterfrom
nsouzaco:patch-1
Feb 12, 2019
Merged

Conditional Rendering translation#51
WendellAdriel merged 14 commits into
reactjs:masterfrom
nsouzaco:patch-1

Conversation

@nsouzaco

@nsouzaco nsouzaco commented Feb 9, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@nsouzaco nsouzaco changed the title translation first version Conditional Rendering translation Feb 9, 2019
Comment thread content/docs/conditional-rendering.md Outdated
@fjoshuajr fjoshuajr added the needs review A pull request ready to be reviewed label Feb 9, 2019
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
fix url

Co-Authored-By: cstpk <costasnataly@gmail.com>
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
fix typos, accents and links

Co-Authored-By: cstpk <costasnataly@gmail.com>

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

Check the comments, please.

Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
@fjoshuajr fjoshuajr 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

Copy link
Copy Markdown
Contributor

@cstpk

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

fjoshuajr and others added 8 commits February 11, 2019 11:19
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
@nsouzaco

Copy link
Copy Markdown
Contributor Author

@cstpk

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

Oi @glaucia86 , conclui as mudancas mas nao estou conseguindo alterar a label.

fjoshuajr
fjoshuajr previously approved these changes Feb 11, 2019

@fjoshuajr fjoshuajr 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 👌

@cstpk Somente os membros podem alterar a label.

@fjoshuajr fjoshuajr added needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input labels Feb 11, 2019
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
Comment thread content/docs/conditional-rendering.md Outdated
@WendellAdriel

Copy link
Copy Markdown
Contributor

Apenas algumas pequenas alterações.

@glaucia86

Copy link
Copy Markdown
Contributor

@cstpk faz as alterações que o @WendellAdriel propôs aqui e nos sinalize ;)

@glaucia86 glaucia86 added needs author response Changes requested needing author's input and removed needs +1 Needs another approval before merging labels Feb 12, 2019
Co-Authored-By: cstpk <costasnataly@gmail.com>
WendellAdriel and others added 2 commits February 11, 2019 20:11
Co-Authored-By: cstpk <costasnataly@gmail.com>
Co-Authored-By: cstpk <costasnataly@gmail.com>
@nsouzaco

Copy link
Copy Markdown
Contributor Author

concluido!

@fjoshuajr fjoshuajr added feedback addressed and removed needs author response Changes requested needing author's input labels Feb 12, 2019

@WendellAdriel WendellAdriel 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!!!

@WendellAdriel
WendellAdriel merged commit 256a879 into reactjs:master Feb 12, 2019
@WendellAdriel

Copy link
Copy Markdown
Contributor

Merge feito!!!
Muito obrigado pela contribuição @cstpk!!! 😉

@WendellAdriel WendellAdriel added 🌐 translated Translation is completed for this page and removed feedback addressed labels Feb 12, 2019
devin-ai-integration Bot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
* translation first version

* Update content/docs/conditional-rendering.md

fix url

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Apply suggestions from code review

fix typos, accents and links

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* removed extra new line

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>

* Update content/docs/conditional-rendering.md

Co-Authored-By: cstpk <costasnataly@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌐 translated Translation is completed for this page

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants