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

docs: Translate add-seo-component.md #118

Merged
merged 4 commits into from
Jan 7, 2020
Merged

Conversation

jpedroschmitz
Copy link
Contributor

@jpedroschmitz jpedroschmitz commented Dec 14, 2019

Qual o objetivo dessa pull request?

  • Adição de uma nova tradução

Qual arquivo foi traduzido/corrigido?

Site
Repositório

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

Fiquei com um pouco de dificuldade de traduzir a palavra alias na documentação, quando precisamos renomear os dados da consulta. Se alguém tiver uma outra sugestão para essa parte eu agradeço:

The next step is to destructure the data from the query and to create an object that checks if the props were used — if not the default values are utilized. The name aliasing comes in handy here: It avoids name collisions.

Como pensei em traduzir:

A próxima etapa é desestruturar os dados da consulta e criar um objeto que verifique se os objetos foram usados - se não forem utilizados os valores padrão. Renomear os itens da consulta é útil aqui: evita colisões de dados.

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 com a comunidade Gatsby @jpedroschmitz. Recomendo dar uma olhada nas correções que eu coloquei abaixo.

@jpedroschmitz
Copy link
Contributor Author

Obrigado por contribuir com a comunidade Gatsby @jpedroschmitz. Recomendo dar uma olhada nas correções que eu coloquei abaixo.

Sempre um prazer poder ajudar essa comunidade incrível!

E obrigado pelas sugestões, foram muito válidas 💜

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 @jpedroschmitz, listei alguns pontos, veja o que acha. ⚔️

Copy link
Contributor

@gabrielferreiraa gabrielferreiraa 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 tradução @jpedroschmitz 🎉 🎉
Deixei algumas considerações 😄


- [marisamorby.com](https://github.com/marisamorby/marisamorby.com/blob/master/packages/gatsby-theme-blog-sanity/src/components/seo.js)
- [gatsby-starter-prismic](https://github.com/LeKoArts/gatsby-starter-prismic/blob/master/src/components/SEO/SEO.jsx)

As mentioned at the beginning you are also able to use the component in templates, like in [this example](https://github.com/jlengstorf/marisamorby.com/blob/6e86f845185f9650ff95316d3475bb8ac86b15bf/src/templates/post.js#L12-L18).
Conforme mencionado no começo, você também pode usar o componente em modelos em templates, como [no exemplo](https://github.com/jlengstorf/marisamorby.com/blob/6e86f845185f9650ff95316d3475bb8ac86b15bf/src/templates/post.js#L12-L18).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Conforme mencionado no começo, você também pode usar o componente em modelos em templates, como [no exemplo](https://github.com/jlengstorf/marisamorby.com/blob/6e86f845185f9650ff95316d3475bb8ac86b15bf/src/templates/post.js#L12-L18).
Conforme mencionado no começo, você também pode usar o componente em modelos em templates, conforme [neste exemplo](https://github.com/jlengstorf/marisamorby.com/blob/6e86f845185f9650ff95316d3475bb8ac86b15bf/src/templates/post.js#L12-L18).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Acho que deixando dois conforme fica um pouco estranho.
Talvez trocar o no por um neste, o que você acha?

Conforme mencionado no começo, você também pode usar o componente em modelos, como neste exemplo.

Copy link
Contributor

Choose a reason for hiding this comment

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

Acho top! Pode ser.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alterado! 🚀


Now define the query and place it in the StaticQuery (you can also save the query in a constant). You can also alias query items, so `title` gets renamed to `defaultTitle`.
Agora defina a consulta (query) e coloque-a na StaticQuery (você também pode salvar a consulta em uma constante). Além disso, você poderia renomear os itens da consulta, para que, por exemplo, `title` utilize o alias de `defaultTitle`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Precisávamos pensar num nome legal para colocar no lugar de alias 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pois é, eu não consegui pensar em nada.

Copy link
Contributor

@iaurg iaurg Dec 22, 2019

Choose a reason for hiding this comment

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

Ví algumas traduções de "alias" como "pseudônimo", poderíamos dar a ideia de um "novo nome" ao title -> defaultTitle. Além disso, você poderia renomear os itens da consulta, para que, por exemplo, title utilize o nome/pseudônimo de defaultTitle ou Além disso, você poderia renomear os itens da consulta, para que, por exemplo, title possa ser utilizado com o nome defaultTitle

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.

Parece que está tudo nos conformes, por mim ta aprovado. Agora é só esperar os outros revisores aprovarem tb

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 🚀

@iaurg iaurg merged commit 23aa091 into master Jan 7, 2020
@jessescn jessescn deleted the traducao-add-seo-component branch April 22, 2020 19:34
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.

4 participants