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 2016-01-08-A-implies-B-does-not-imply-B-implies-A.md #353

Merged
merged 2 commits into from Nov 12, 2019
Merged

Translate 2016-01-08-A-implies-B-does-not-imply-B-implies-A.md #353

merged 2 commits into from Nov 12, 2019

Conversation

mdapper
Copy link
Contributor

@mdapper mdapper commented Oct 28, 2019

Tradução do blog post 2016-01-08-A-implies-B-does-not-imply-B-implies-A.md

@mdapper mdapper mentioned this pull request Oct 28, 2019
Closed
45 tasks
@netlify
Copy link

netlify bot commented Oct 28, 2019 •
edited

Deploy preview for pt-br-reactjs ready!

Built with commit f1992e3

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

@jhonmike jhonmike added blog Blog post translation needs review A pull request ready to be reviewed labels Oct 28, 2019
jhonmike
jhonmike reviewed Nov 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.

Parabéns pela tradução, ao revisar encontrei apenas um errinho no nome da variável.

content/blog/2016-01-08-A-implies-B-does-not-imply-B-implies-A.md Outdated Show resolved Hide resolved
@jhonmike jhonmike added needs author response Changes requested needing author's input and removed needs review A pull request ready to be reviewed labels Nov 8, 2019
@mdapper
Update content/blog/2016-01-08-A-implies-B-does-not-imply-B-implies-A.md
f1992e3 
Co-Authored-By: Jhon Mike <developer@jhonmike.com.br>
@jhonmike jhonmike added needs +1 Needs another approval before merging and removed needs author response Changes requested needing author's input labels Nov 11, 2019
@mdapper
Copy link
Contributor Author

mdapper commented Nov 11, 2019

@mjr e @lucianomlima se puderem, façam o review dessa PR por favor. 😄

mjr
mjr approved these changes Nov 11, 2019
View reviewed changes
Copy link
Contributor

@mjr mjr left a comment

Choose a reason for hiding this comment

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

@mdapper a sua tradução ficou muito boa... 👏. Não encontrei nenhum ajuste ou sugestão para ser feito.

@jhonmike jhonmike added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging labels Nov 12, 2019
@jhonmike jhonmike merged commit 7176f85 into reactjs:master Nov 12, 2019
@mdapper mdapper deleted the translate/A-implies-B-does-not-imply-B-implies-A branch November 12, 2019 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhonmike jhonmike jhonmike approved these changes

@mjr mjr mjr 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
@mdapper @jhonmike @mjr