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

translate: path-prefix.md #65

Merged
merged 3 commits into from
Nov 28, 2019

Conversation

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

path-prefix.md

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

Sugestões:

  1. Chamar path de rota e não de caminho. No contexto de web development chamamos de rotas as URLs que o usuário interage na sua aplicação. Por exemplo, rota de usuário (/user), rota do blog (/blog).
  2. assets a tradução para pt-BR seria "ativo", mas não faz sentido no contexto de web, acho que comunidade já está acostumada a referenciar como assets.
  3. helper apesar de já ser uma palavra popular pode ser chamado de função auxiliar.

@peaonunes peaonunes added the /docs File refering to /docs directory label Nov 22, 2019
@peaonunes peaonunes self-assigned this Nov 22, 2019
@jessescn
Copy link
Contributor

Em relação ao seu comentário sobre path, pelo que eu já vivenciei normalmente traduzem pra rota quando é no contexto de APIs e paths de URL, mas comumente eu vejo traduzir pra caminho quando se refere a posição de um arquivo dentro de um diretório, por exemplo

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 tradução @peaonunes. Só da uma olhadinha nesses pontos que eu listei que acho que são importantes


In addition, links to various resources (JavaScript, CSS, images, and other static content) need the same prefix, so that the site continues to function correctly when served with the path prefix in place.
Além disso, links para vários recursos (JavaScript, CSS, imagens, e outros conteúdos estáticos) precisam do mesmo prefixo para que o website continue a funcionar corretamente quando servido com a rota e prefixo no lugar.
Copy link
Contributor

Choose a reason for hiding this comment

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

"served with the path prefix in place" a tradução ficou um pouco fora de contexto ou pode ser impressão minha, o que tu acha?

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 o que ele quer dizer é que esses recursos estáticos pra funcionar corretamente precisam estar com o mesmo prefixo do site quando o site for servido com esse prefixo na url. Por exemplo uma tag img referenciando um asset de img, usa o path, por isso ele diz que para o site continuar funcionando todos esses recursos estáticos precisam do mesmo prefixo do site em si.
Com served with the path prefix in place acho que ele quer dizer que o site está sendo servido com o prefixo agora. Que tal a seguinte tradução:

quando servido com o prefixo adicionado.

Co-Authored-By: Jessé Souza <[email protected]>
@peaonunes peaonunes requested a review from jessescn November 25, 2019 03:20
jessescn
jessescn previously approved these changes Nov 25, 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.

Muito obrigado pela cotribuição @peaonunes, por mim ta aprovado, agora só esperar outro codeowner aprovar pra poder ser mergeada 🎉 🎉

Copy link
Contributor

@rafaeelaudibert rafaeelaudibert left a comment

Choose a reason for hiding this comment

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

Muito obrigado peaonunes. Apenas levantei uma questãozinha ali pra discutirmos. Se achar que está bem assim, estou ok com isso, e aceitarei o merge. Obrigado novamente! ✨

Co-Authored-By: Rafael Baldasso Audibert <[email protected]>
@jay-jlm jay-jlm changed the title translats path-prefix.md translate: path-prefix.md Nov 27, 2019
Copy link
Contributor

@rafaeelaudibert rafaeelaudibert left a comment

Choose a reason for hiding this comment

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

Ótimo! Obrigado pela contribuição, @peaonunes! Espero que possa contribuir novamente com a gente no futuro. 🎉✨

@rafaeelaudibert rafaeelaudibert merged commit 5475bff into gatsbyjs:master Nov 28, 2019
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