Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

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.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
/docs File refering to /docs directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants