Skip to content

Translate Create a new React App page - #18

Merged
cezaraugusto merged 6 commits into
reactjs:masterfrom
mrcelo:criar-novo-app-react
Feb 9, 2019
Merged

Translate Create a new React App page#18
cezaraugusto merged 6 commits into
reactjs:masterfrom
mrcelo:criar-novo-app-react

Conversation

@mrcelo

@mrcelomrcelo commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

@mrcelomrcelo changed the title Add translation for Create a new React App pageTranslate Create a new React App pageFeb 8, 2019
@mrcelomrcelo mentioned this pull request Feb 8, 2019
Closed
90 tasks
@fjoshuajrfjoshuajr added the needs review A pull request ready to be reviewed label Feb 8, 2019
klzns
klzns suggested changes Feb 8, 2019
View reviewed changes
Comment threadcontent/docs/create-a-new-react-app.md Outdated
Comment threadcontent/docs/create-a-new-react-app.md
Comment threadcontent/docs/create-a-new-react-app.md Outdated
Comment threadcontent/docs/create-a-new-react-app.md
Comment threadcontent/docs/create-a-new-react-app.md Outdated
Comment threadcontent/docs/create-a-new-react-app.md Outdated
Comment threadcontent/docs/create-a-new-react-app.md Outdated
glaucia86
glaucia86 suggested changes Feb 9, 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.

@mrcelo algumas alterações a serem feitas.
Quando terminar colocar como 'Resolve Conversation' que analisaremos novamente as alterações e logo aprovação da sua contribuição. Muito obrigada.

klznsand others added 2 commits February 8, 2019 21:18
@klzns@mrcelo
Fix pronoun
9c94c96
Co-Authored-By: mrcelo <mrcelo@users.noreply.github.com>
@klzns@mrcelo
Fix noun
82d7ef8
Co-Authored-By: mrcelo <mrcelo@users.noreply.github.com>
glaucia86
glaucia86 approved these changes Feb 9, 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: @WendellAdriel@cezaraugusto could you see if it's okay?

@glaucia86glaucia86 added needs resolution Something not defined that needs maintainers input needs +1 Needs another approval before merging and removed needs review A pull request ready to be reviewed labels Feb 9, 2019
cezaraugusto
cezaraugusto reviewed Feb 9, 2019
View reviewed changes

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

nice work @mrcelo! left some nits but overall this looks 👌

Comment threadcontent/docs/create-a-new-react-app.md Outdated
Comment threadcontent/docs/create-a-new-react-app.md Outdated
@cezaraugustocezaraugusto added needs author response Changes requested needing author's input and removed needs +1 Needs another approval before merging needs resolution Something not defined that needs maintainers input labels Feb 9, 2019
cezaraugusto
cezaraugusto approved these changes Feb 9, 2019
View reviewed changes

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

thanks @mrcelo! way to go!

@cezaraugusto
cezaraugusto merged commit af17c4e into reactjs:masterFeb 9, 2019
@cezaraugustocezaraugusto added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Feb 9, 2019
devin-ai-integrationBot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@mrcelo@cezaraugusto
Translate Create a new React App page (reactjs#18)
9052986
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers
@klznsklznsklzns requested changes
@glaucia86glaucia86glaucia86 approved these changes
@cezaraugustocezaraugustocezaraugusto 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

@mrcelo@klzns@glaucia86@cezaraugusto@fjoshuajr