-
Notifications
You must be signed in to change notification settings - Fork 100
Conversation
There was a problem hiding this 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.
Co-Authored-By: Jessé Souza <[email protected]>
Sempre um prazer poder ajudar essa comunidade incrível! E obrigado pelas sugestões, foram muito válidas 💜 |
There was a problem hiding this 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. ⚔️
There was a problem hiding this 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 😄
docs/docs/add-seo-component.md
Outdated
|
||
- [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). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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). |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho top! Pode ser.
There was a problem hiding this comment.
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`. |
There was a problem hiding this comment.
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
🤔
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
Co-Authored-By: Italo <[email protected]>
There was a problem hiding this 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
There was a problem hiding this 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 🚀
Qual o objetivo dessa pull request?
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:Como pensei em traduzir: