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

Traduzido graphql-concepts.md #96

Merged
merged 2 commits into from
Dec 16, 2019
Merged

Traduzido graphql-concepts.md #96

merged 2 commits into from
Dec 16, 2019

Conversation

DouglasMV
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?

graphql-concepts.md

Algum comentário em relação a tradução?

iaurg
iaurg suggested changes Dec 10, 2019
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.

Boa tradução, apontei apenas alguns pontos para análise. Veja o que acha.

docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
docs/docs/graphql-concepts.md Outdated Show resolved Hide resolved
@DouglasMV
Copy link
Contributor Author

Boa tradução, apontei apenas alguns pontos para análise. Veja o que acha.

Bem apontadas as correçoes, fiz um novo commit corrigindo todos esses pontos e mais alguns lugares que eu havia traduzido a palavra schema.

@jessescn jessescn added the /docs File refering to /docs directory label Dec 10, 2019
@jessescn jessescn mentioned this pull request Dec 10, 2019
Open
iaurg
iaurg approved these changes Dec 14, 2019
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 aceitar as sugestões! Ficou muito boa a tradução 🎉

jessescn
jessescn approved these changes Dec 16, 2019
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.

Aprovado! Obrigado por contribuir conosco @DouglasMV!

@jessescn jessescn merged commit c6e2ea6 into gatsbyjs:master Dec 16, 2019
@DouglasMV DouglasMV mentioned this pull request Dec 20, 2019
Merged
2 tasks
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

@abnersajr abnersajr abnersajr 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.
4 participants
@DouglasMV @abnersajr @iaurg @jessescn