Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Translate Passing Functions to Components #16

Merged
merged 2 commits into from Feb 17, 2019
Merged

Translate Passing Functions to Components #16

merged 2 commits into from Feb 17, 2019

Conversation

negreirosleo
Copy link
Contributor

@negreirosleo negreirosleo commented Feb 8, 2019

#1

Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

@negreirosleo precisa fazer algumas alterações antes de aprovarmos.
Pedimos que, veja as sugestões sinalizadas e quando for acertando, marque como 'Resolve conversation'
E depois nos sinalize de que concluiu as alterações.

Assim que tiver pelo menos dois code reviews aprovados, aprovaremos a sua contribuição.
Muito obrigada! :)

@glaucia86 glaucia86 added needs review A pull request ready to be reviewed needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Feb 8, 2019
@negreirosleo
Copy link
Contributor Author

@glaucia86 Irei fazer as alterações, muito obrigado.

@glaucia86
Copy link
Member

@negreirosleo

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

Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

@negreirosleo

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

content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
content/docs/faq-functions.md Outdated Show resolved Hide resolved
@glaucia86
Copy link
Member

@negreirosleo precisa de alguma ajuda com essa PR's? Se você ver que não conseguirá dar conta, nos sinalize por favor ;)

@negreirosleo
Copy link
Contributor Author

@negreirosleo precisa de alguma ajuda com essa PR's? Se você ver que não conseguirá dar conta, nos sinalize por favor ;)

Opa, desculpa a demora @glaucia86, tive uma semana um tanto quanto cheia e não pude me dedicar tanto à branch, mas estou acertando todas as sugestões de mudança agora 👍

@negreirosleo
Copy link
Contributor Author

@negreirosleo

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!

Eu não sei aonde altero a label :(

@halian-vilela
Copy link
Collaborator

Eu não sei aonde altero a label :(

@negreirosleo

Alterei para você! E fiz algumas sugestões ali na conversa que não tinha ficado resolvida, dê uma olhada!

Abraços!

Copy link
Member

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

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

visto que todos os comentarios foram resolvidos, aprovando

@cezaraugusto cezaraugusto merged commit c9fa437 into reactjs:master Feb 17, 2019
@cezaraugusto cezaraugusto added 🌐 translated Translation is completed for this page and removed feedback addressed needs author response Changes requested needing author's input labels Feb 17, 2019
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.

None yet

4 participants