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 blog you-probably-dont-need-derived-state #183

Merged
merged 6 commits into from
Apr 8, 2019
Merged

Translate blog you-probably-dont-need-derived-state #183

merged 6 commits into from
Apr 8, 2019

Conversation

Luanf
Copy link
Contributor

@Luanf Luanf commented Mar 23, 2019

Segue PR do blog post "you-probably-dont-need-derived-state".
Agradeço as reviews! 🙏

@Luanf Luanf mentioned this pull request Mar 23, 2019
Closed
45 tasks
@netlify
Copy link

netlify bot commented Mar 23, 2019 •
edited
Loading

Deploy preview for pt-br-reactjs ready!

Built with commit 5834ed7

https://deploy-preview-183--pt-br-reactjs.netlify.com

@jhonmike jhonmike added needs review A pull request ready to be reviewed blog Blog post translation labels Mar 23, 2019
gstvribs
gstvribs reviewed Mar 29, 2019
View reviewed changes
Copy link
Contributor

@gstvribs gstvribs left a comment

Choose a reason for hiding this comment

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

Creio que seja apenas a consideração do parcimônia, de resto está IMHO
Aliás, ótima tradução !

@jhonmike jhonmike added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Mar 29, 2019
@Luanf
Copy link
Contributor Author

Luanf commented Mar 29, 2019

Revisado! Obrigado! 😄

@jhonmike jhonmike added needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input labels Mar 30, 2019
jhonmike
jhonmike reviewed Apr 8, 2019
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.

pequenos ajustes de formatação

content/blog/2018-06-07-you-probably-dont-need-derived-state.md Outdated Show resolved Hide resolved
content/blog/2018-06-07-you-probably-dont-need-derived-state.md Outdated Show resolved Hide resolved
content/blog/2018-06-07-you-probably-dont-need-derived-state.md Outdated Show resolved Hide resolved
content/blog/2018-06-07-you-probably-dont-need-derived-state.md Outdated Show resolved Hide resolved
@jhonmike jhonmike added needs author response Changes requested needing author's input and removed needs +1 Needs another approval before merging labels Apr 8, 2019
@Luanf
Apply suggestions from code review
914bb8c 
Co-Authored-By: Luanf <luanegidioferreira@gmail.com>
@Luanf
Copy link
Contributor Author

Luanf commented Apr 8, 2019

Bons olhos!!! Valeu 😄

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs author response Changes requested needing author's input labels Apr 8, 2019
@jhonmike jhonmike merged commit 9dffe63 into reactjs:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gstvribs gstvribs gstvribs left review comments

@jhonmike jhonmike jhonmike approved these changes

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@Luanf @jhonmike @gstvribs