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.

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

Translate adding-forms.md #93

Merged
merged 5 commits into from
Dec 16, 2019
Merged

Translate adding-forms.md #93

merged 5 commits into from
Dec 16, 2019

Conversation

tavareshenrique
Copy link
Contributor

@tavareshenrique tavareshenrique commented Dec 6, 2019

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?

adding-forms.md
https://github.com/gatsbyjs/gatsby-pt-BR/blob/master/docs/docs/adding-forms.md

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

Eu não substitui o termo "Hello World" por "Olá, Mundo", pois a expressão "Hello World" é mundialmente conhecida pelos programadores, acho que não tem necessidade a sua tradução, a menos que achem o contrário para seguir alguma diretriz da comunidade. Traduzi a palavra "state" para "estado", mas dando ênfase que são a mesma coisa, segui um padrão da documentarão do próprio React. Ao invés de traduzir literal e usar "campo de entrada", mantive "input", acredito que fica de melhor entendimento. Ao final quando é indicado bibliotecas de formulários, além do Formik e do Final Form, eu também acrescentei o Unform da Rocketseat, é uma biblioteca de formularia nova, que vêm sendo utilizando por bastante gente e já se encontra estável, acho legal adiciona-la, caso contrário posso estar removendo.

@jessescn jessescn added the /docs File refering to /docs directory label Dec 7, 2019
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 por contribuir @tavareshenrique! Eu apontei algumas coisas pra resolver (pode parece muito, mas a grande maioria é sugerindo tirar o ;), acho que seria massa se tu desse uma olhada e deixasse tua opinião.

docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
docs/docs/adding-forms.md Outdated Show resolved Hide resolved
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 por contribuir @tavareshenrique! Eu apontei algumas coisas pra resolver (pode parece muito, mas a grande maioria é sugerindo tirar o ;), acho que seria massa se tu desse uma olhada e deixasse tua opinião.

@jessescn
Copy link
Contributor

Creio que ainda esteja faltando tirar alguns ;

@tavareshenrique
Copy link
Contributor Author

Desculpe o equivoco @jessescn, fui traído pelo meu editor hahaha agora sim!

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.

Ótima tradução, obrigado por contribuir 🎉

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 @tavareshenrique. Aprovado!

@jessescn jessescn merged commit d2621bd into gatsbyjs:master Dec 16, 2019
@tavareshenrique
Copy link
Contributor Author

Eu que agradeço pessoal, vamos juntos!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
/docs File refering to /docs directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants