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

New translation: progressive-web-app.md #291

Merged
merged 14 commits into from
Aug 31, 2020

Conversation

benyAlves
Copy link
Contributor

@benyAlves benyAlves commented Aug 6, 2020

Qual o objetivo dessa pull request?

Tradução

  • Adição de uma nova tradução
  • Correção em uma tradução existente

Qual arquivo foi traduzido/corrigido?

docs\docs\progressive-web-app.md

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

@thatfiredev thatfiredev changed the title Traducao progressive web app.md New translation: progressive-web-app.md Aug 6, 2020
@thatfiredev thatfiredev added the /docs File refering to /docs directory label Aug 6, 2020
Copy link
Contributor

@thatfiredev thatfiredev left a comment

Choose a reason for hiding this comment

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

@benyAlves , óptimo trabalho nesta sua primeira contribuição para o Gatsby.

Começar dizendo que não sou o revisor oficial destas traduções. Deixei apenas alguns comentários que talvez possam ajudar os revisores oficiais.

Se você não concorda com alguma sugestão, não faça a alteração sugerida e espere por um revisor oficial que irá decidir se a sugestão faz sentido ou não.

benyAlves and others added 4 commits August 6, 2020 19:27
Manter o termo Progressive web App para melhor percepção

Co-authored-by: Rosário Pereira Fernandes <[email protected]>
Co-authored-by: Rosário Pereira Fernandes <[email protected]>
Co-authored-by: Rosário Pereira Fernandes <[email protected]>
Copy link
Contributor

@henriquepw henriquepw 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 @benyAlves! 🎉

Fiz algumas sugestões de correções ortográficas, poderia da uma olhada?

@benyAlves
Copy link
Contributor Author

Obrigado pela revisão

thatfiredev
thatfiredev previously approved these changes Aug 13, 2020
thatfiredev
thatfiredev previously approved these changes Aug 14, 2020
@thayannevls
Copy link
Contributor

thayannevls commented Aug 16, 2020

Queria apenas fazer uma sugestão de trocar PWA para masculino na tradução(Exemplo: "Os PWAs", "do PWA" etc) porque é o padrão mais adotado nos artigos em português, inclusive nas documentações oficiais da Google.

Excelente tradução! Ao corrigir esse pequeno detalhe, está aprovado por mim

@benyAlves
Copy link
Contributor Author

Obrigado pela revisão @thayannevls
A correção já foi feita

Copy link
Contributor

@thayannevls thayannevls left a comment

Choose a reason for hiding this comment

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

Faltou só esse 👀

Copy link
Contributor

@thayannevls thayannevls left a comment

Choose a reason for hiding this comment

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

Obrigada pela contribuição 🚀

@thayannevls thayannevls merged commit daab7dc into gatsbyjs:master Aug 31, 2020
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