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: 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

docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved

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.

docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
Co-Authored-By: Jessé Souza <[email protected]>
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! ✨

docs/docs/path-prefix.md Outdated Show resolved Hide resolved
docs/docs/path-prefix.md Outdated Show resolved Hide resolved
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 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.

4 participants