Skip to content

Conversation

augustodg2
Copy link
Contributor

@augustodg2 augustodg2 commented Apr 29, 2023

Issue #555

This PR translates the page "You Might Not Need an Effect" to pt-BR

t0in
t0in previously approved these changes May 2, 2023
Copy link
Contributor

@TarsiSurdi TarsiSurdi left a comment

Choose a reason for hiding this comment

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

Fiz diversas melhorias à tradução, dentre elas a maior parte se resume a conformar ao GLOSSARY com a tradução de termos comuns como state, prop ou event handler. No geral a página está ótima e de fácil entendimento em nosso idioma, parabéns!

TarsiSurdi

This comment was marked as duplicate.

@augustodg2 augustodg2 requested a review from TarsiSurdi May 5, 2023 14:57
TarsiSurdi
TarsiSurdi previously approved these changes May 7, 2023
@augustodg2
Copy link
Contributor Author

Obrigado pela review @TarsiSurdi, acredito que resolvi todas as questões levantadas.

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs review A pull request ready to be reviewed labels May 10, 2023
@jhonmike jhonmike merged commit bae363a into reactjs:main May 10, 2023
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.

4 participants