Skip to content

Rules / Hooks reference - #76

Merged
fjoshuajr merged 14 commits into
reactjs:masterfrom
thadeu:hooks-reference
Feb 14, 2019
Merged

Rules / Hooks reference#76
fjoshuajr merged 14 commits into
reactjs:masterfrom
thadeu:hooks-reference

Conversation

@thadeu

@thadeuthadeu commented Feb 11, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@WendellAdrielWendellAdriel added the needs review A pull request ready to be reviewed label Feb 11, 2019
glaucia86
glaucia86 suggested 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.

@thadeu

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.

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

thadeu commented Feb 12, 2019

Copy link
Copy Markdown
ContributorAuthor

@glaucia86 tudo certo.. muito obrigado! No entanto, me parece que o texto de referência deste projeto ao Hooks Reference, não está totalmente atualizado.

@eduardomoronieduardomoroni added needs +1 Needs another approval before merging and removed needs review A pull request ready to be reviewed needs author response Changes requested needing author's input labels Feb 12, 2019
@cezaraugustocezaraugusto added needs rebase and removed needs +1 Needs another approval before merging labels Feb 13, 2019
@cezaraugusto

cezaraugusto commented Feb 13, 2019

Copy link
Copy Markdown
Contributor

@thadeu pode fazer um rebase?

@thadeu

thadeu commented Feb 13, 2019

Copy link
Copy Markdown
ContributorAuthor

@thadeu pode fazer um rebase?

@cezaraugusto Sim! claro.

@fjoshuajrfjoshuajr added 💪 long page - more than 100 lines Long page needs review A pull request ready to be reviewed labels Feb 13, 2019
@thadeu

thadeu commented Feb 14, 2019

Copy link
Copy Markdown
ContributorAuthor

@cezaraugusto no conflicts in the branch.

glaucia86
glaucia86 approved these changes Feb 14, 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.

@thadeu

LGTM 👍
Muito obrigada pela sua contribuição!

cc: @fjoshuajr@WendellAdriel@eduardomoroni@cezaraugusto só aprovar.

@glaucia86glaucia86 added needs +1 Needs another approval before merging and removed needs review A pull request ready to be reviewed 💪 long page - more than 100 lines Long page labels Feb 14, 2019
@fjoshuajrfjoshuajr added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging needs rebase labels Feb 14, 2019
fjoshuajr
fjoshuajr approved these changes Feb 14, 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.

🎉🎉

@fjoshuajr
fjoshuajr merged commit ce6c5b4 into reactjs:masterFeb 14, 2019
@tesseralistesseralis mentioned this pull request Feb 14, 2019
Closed
90 tasks
@halian-vilelahalian-vilela mentioned this pull request Feb 21, 2019
Closed
8 tasks
devin-ai-integrationBot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@thadeu@fjoshuajr
Rules / Hooks reference (reactjs#76)
bfd181b
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@glaucia86glaucia86glaucia86 approved these changes
@fjoshuajrfjoshuajrfjoshuajr approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@glaucia86glaucia86

Labels

🌐 translatedTranslation is completed for this page

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@thadeu@cezaraugusto@glaucia86@fjoshuajr@eduardomoroni@WendellAdriel