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 answering-it-security.md #161

Merged
merged 3 commits into from
Jan 24, 2020
Merged

Translate answering-it-security.md #161

merged 3 commits into from
Jan 24, 2020

Conversation

mauriciomutte
Copy link
Contributor

Qual o objetivo dessa pull request?

  • Adição de uma nova tradução
  • Correção em uma tradução existente

Qual arquivo foi traduzido/corrigido?

answering-it-security.md

@mauriciomutte mauriciomutte added the /docs File refering to /docs directory label Jan 17, 2020
jessescn
jessescn suggested changes Jan 17, 2020
View reviewed changes
Copy link
Contributor

@jessescn jessescn left a comment

Choose a reason for hiding this comment

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

Apenas duas besteiras, de resto ta ótimo!

docs/docs/answering-it-security.md Outdated Show resolved Hide resolved
docs/docs/answering-it-security.md Outdated Show resolved Hide resolved
@jessescn jessescn mentioned this pull request Jan 17, 2020
Open
iaurg
iaurg suggested changes Jan 18, 2020
View reviewed changes
Copy link
Contributor

@iaurg iaurg left a comment

Choose a reason for hiding this comment

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

Muito boa a tradução, apenas apontei um detalhe.

docs/docs/answering-it-security.md Outdated Show resolved Hide resolved
@mauriciomutte
Copy link
Contributor Author

Obrigado pelas correções @jessescn e @iaurg!
Ambas foram devidamente corrigidas 👍

iaurg
iaurg approved these changes Jan 21, 2020
View reviewed changes
Copy link
Contributor

@iaurg iaurg left a comment

Choose a reason for hiding this comment

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

Obrigado por contribuir Maurício, por mim está tudo certo. 🕺

jessescn
jessescn approved these changes Jan 24, 2020
View reviewed changes
Copy link
Contributor

@jessescn jessescn left a comment

Choose a reason for hiding this comment

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

Obrigado pela contribuição @mauriciomutte. 🎉

@jessescn jessescn merged commit ffec634 into gatsbyjs:master Jan 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jessescn jessescn jessescn approved these changes

@iaurg iaurg iaurg approved these changes

@dennissiq dennissiq Awaiting requested review from dennissiq

@jay-jlm jay-jlm Awaiting requested review from jay-jlm

@rafaeelaudibert rafaeelaudibert Awaiting requested review from rafaeelaudibert

Assignees
No one assigned
Labels
/docs File refering to /docs directory
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mauriciomutte @iaurg @jessescn