-
Notifications
You must be signed in to change notification settings - Fork 100
New translation: progressive-web-app.md #291
New translation: progressive-web-app.md #291
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.
@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.
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]>
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 pela tradução @benyAlves! 🎉
Fiz algumas sugestões de correções ortográficas, poderia da uma olhada?
Co-authored-by: Henrique Miranda <[email protected]>
Co-authored-by: Henrique Miranda <[email protected]>
Co-authored-by: Henrique Miranda <[email protected]>
Co-authored-by: Henrique Miranda <[email protected]>
Co-authored-by: Henrique Miranda <[email protected]>
Obrigado pela revisão |
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 |
Obrigado pela revisão @thayannevls |
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.
Faltou só esse 👀
Co-authored-by: Thayanne Luiza <[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.
Obrigada pela contribuição 🚀
Qual o objetivo dessa pull request?
Tradução
Qual arquivo foi traduzido/corrigido?
docs\docs\progressive-web-app.md
Algum comentário em relação a tradução?