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.

Sign up for GitHub

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

Jump to bottom

issue #555 - traducao da pagina start-a-new-react-project #775

Merged
merged 10 commits into from
Nov 1, 2024
Merged

issue #555 - traducao da pagina start-a-new-react-project #775

merged 10 commits into from
Nov 1, 2024

Conversation

Thales-Coutinho
Copy link
Contributor

Essa PR termina de traduzir a página /learn/start-a-new-react-project

#555

@vercel Vercel
Copy link

vercel bot commented Aug 15, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
pt-br-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Aug 25, 2024 0:59am

MateusCaltabiano
MateusCaltabiano suggested changes Aug 19, 2024
View reviewed changes
Copy link
Contributor

@MateusCaltabiano MateusCaltabiano left a comment

Choose a reason for hiding this comment

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

Boa tradução! Acredito que o único "erro" mesmo seria a não tradução de "Framework", que já está sendo traduzido no topo da página. Fora isso, são sugestões para puxar a tradução mais para o natural ao invés do literal.

src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
@Thales-Coutinho
Copy link
Contributor Author

Gostei das sugestões, elas mantem o estilo do que havia antes e corrige alguns errinhos que tinha cometido, obrigado!

MateusCaltabiano
MateusCaltabiano approved these changes Aug 22, 2024
View reviewed changes
Copy link
Contributor

@MateusCaltabiano MateusCaltabiano left a comment

Choose a reason for hiding this comment

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

LGTM!

jlucfarias
jlucfarias suggested changes Aug 24, 2024
View reviewed changes
Copy link
Contributor

@jlucfarias jlucfarias left a comment

Choose a reason for hiding this comment

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

Fora a alteração necessária, está tudo tranquilo com a tradução

src/content/learn/start-a-new-react-project.md Outdated Show resolved Hide resolved
jhonmike
jhonmike approved these changes Nov 1, 2024
View reviewed changes
Copy link
Member

@jhonmike jhonmike left a comment

Choose a reason for hiding this comment

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

Thank you very much for the correction to an existing translation, it really helps us to keep the pages consistent with the original document!!

Thanks again!

@jhonmike jhonmike added the 🌐 translated Translation is completed for this page label Nov 1, 2024
@jhonmike jhonmike merged commit 034a591 into reactjs:main Nov 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MateusCaltabiano MateusCaltabiano MateusCaltabiano approved these changes

@jlucfarias jlucfarias jlucfarias approved these changes

@jhonmike jhonmike jhonmike approved these changes

Assignees
No one assigned
Labels
🌐 translated Translation is completed for this page
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Thales-Coutinho @jhonmike @jlucfarias @MateusCaltabiano