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

Translate updating-objects-in-state.md #679

Merged
merged 2 commits into from
May 31, 2023
Merged

Translate updating-objects-in-state.md #679

merged 2 commits into from
May 31, 2023

Conversation

Egpereira
Copy link
Contributor

Tradução do arquivo updating-objects-in-state.md

Dúvidas de tradução

Algumas dúvidas durante a tradução sobre alguns termos que gostaria de segundas opiniões, se possível:

  • spread syntax -> sintaxe de espalhamento (seguindo o termo utilizado pela mdn web docs)
  • snapshot -> mantido como snapshots
  • read-only -> somente leitura
  • without additional performance or correctness pitfalls. -> sem armadilhas adicionais de desempenho ou corretude

Links adicionais

@TarsiSurdi
Copy link
Contributor

Concordo com a tradução de spread syntax como sintaxe de espalhamento, assim como feito na #676.
Quanto ao snapshot fiquei bastante em dúvida nas traduções que fiz pois sempre interpretei como "um momento isolado no tempo". Talvez seja um estrangeirismo o qual teremos que incorporar na documentação ou deve-se estabelecer alguma guideline sobre como o traduzir no GLOSSARY.

@jhonmike jhonmike mentioned this pull request May 2, 2023
Open
@jhonmike jhonmike added the needs review A pull request ready to be reviewed label May 2, 2023
dieguesmosken
dieguesmosken suggested changes May 29, 2023
View reviewed changes
Copy link

@dieguesmosken dieguesmosken left a comment

Choose a reason for hiding this comment

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

sugiro pequenas mudanças na tradução onde comentei anteriormente

src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
@Egpereira Egpereira force-pushed the translate-updating-objects-in-state branch from ac68125 to 8da1f32 Compare May 30, 2023 14:32
dieguesmosken
dieguesmosken approved these changes May 30, 2023
View reviewed changes
Copy link

@dieguesmosken dieguesmosken left a comment

Choose a reason for hiding this comment

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

a tradução está coesa, sem erros

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs review A pull request ready to be reviewed labels May 31, 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 077c26a into reactjs:main May 31, 2023
@Egpereira Egpereira deleted the translate-updating-objects-in-state branch May 31, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dieguesmosken dieguesmosken dieguesmosken 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
@Egpereira @TarsiSurdi @jhonmike @dieguesmosken