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 add-react-to-an-existing-project #659

Merged
merged 16 commits into from
Apr 28, 2023
Merged

Conversation

revogabe
Copy link
Contributor

In this PR I am translating the page that I was designated called "add-react-to-an-existing-project" I also saw that in the open issue this page is described with the old name.

In this PR I am translating the page that I was designated called "add-react-to-an-existing-project" I also saw that in the open issue this page is described with the old name.
---

<Intro>

If you want to add some interactivity to your existing project, you don't have to rewrite it in React. Add React to your existing stack, and render interactive React components anywhere.
Se você quer adicionar alguma interatividade ao seu projeto existente, você não precisa reescrevê-lo em React. Adicione React à sua pilha existente e renderize componentes React interativos em qualquer lugar.
Copy link
Contributor

Choose a reason for hiding this comment

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

A tradução de stack pra pilha acaba não fazendo muito sentido, também achei a sintaxe da segunda frase levemente confusa.

Sugestão:

Se você quer adicionar alguma interatividade ao seu projeto existente, você não precisa reescrevê-lo em React. Adicione React à sua aplicação e renderize componentes interativos em qualquer lugar utilizando React.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Obrigado irei dar uma olhada novamente na tradução

Copy link
Member

Choose a reason for hiding this comment

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

eu usuária stack mesmo 🤔


</Intro>

<Note>

**You need to install [Node.js](https://nodejs.org/en/) for local development.** Although you can [try React](/learn/installation#try-react) online or with a simple HTML page, realistically most JavaScript tooling you'll want to use for development requires Node.js.
**Você precisa instalar o [Node.js](https://nodejs.org/en/) para o desenvolvimento local.** Embora você possa [experimentar o React](/learn/installation#try-react) online ou com uma simples página HTML, realisticamente a maioria das ferramentas JavaScript que você vai querer usar para desenvolvimento requer o Node.js.
Copy link
Contributor

Choose a reason for hiding this comment

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

Apenas uma pequena correção: "requer" concorda com "ferramentas", então precisa estar no plural também, "requerem".

Copy link
Contributor

Choose a reason for hiding this comment

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

Na verdade, aqui o sujeito inicia-se com "a maioria" eu pesquisei esse caso de concordância e fiquei chocado que "requer" tá correto.

Talvez dar uma reorganizada na frase pra utilizarmos o "requerem", mas surpreendentemente da forma que tá escrita assim, está correto.

Copy link
Contributor

Choose a reason for hiding this comment

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

"requer" está concordando com "a maioria", está ok

Copy link
Contributor

@natrodrigo natrodrigo left a comment

Choose a reason for hiding this comment

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

@revogabe obrigado pela contribuição. A tradução está ótima!

@jhonmike jhonmike added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Apr 27, 2023
@jhonmike
Copy link
Member

Tenta rodar local para ver porque o build esta quebrando

Co-authored-by: Natã Pereira <[email protected]>
@revogabe
Copy link
Contributor Author

Done! 🥳

@jhonmike jhonmike added needs review A pull request ready to be reviewed needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input needs review A pull request ready to be reviewed labels Apr 27, 2023
@jhonmike jhonmike added the needs author response Changes requested needing author's input label Apr 27, 2023
Co-authored-by: Felipe Freitag Vargas <[email protected]>
revogabe and others added 2 commits April 27, 2023 11:36
Co-authored-by: Felipe Freitag Vargas <[email protected]>
Co-authored-by: Felipe Freitag Vargas <[email protected]>
Co-authored-by: Felipe Freitag Vargas <[email protected]>
@revogabe
Copy link
Contributor Author

Done ! 🥳

@revogabe
Copy link
Contributor Author

Obrigado não tinha me atentado a estes links, desculpe estar tendo que fazer bastante mudanças, mas estou bem feliz por estar contribuindo com a tradução! 😅

@jhonmike
Copy link
Member

jhonmike commented Apr 27, 2023

Obrigado não tinha me atentado a estes links, desculpe estar tendo que fazer bastante mudanças, mas estou bem feliz por estar contribuindo com a tradução! 😅

Rapaz tradução é assim mesmo, na real quando mais sugestões melhor fica a doc, ta showw!!

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Apr 28, 2023
@github-actions
Copy link

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@jhonmike jhonmike merged commit 6e32e0e into reactjs:main Apr 28, 2023
@revogabe
Copy link
Contributor Author

revogabe commented Apr 28, 2023

Thank you!! 🥳 Lets GO!!

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.

6 participants