I18N - Home

[MoisesMsa]

Português (BR) | English (US)

Comunidade

• Eu li e segui o Guia de Contribuição.
• Eu li e segui o Código de Conduta.

Tipo de alteração

• 🐞 Correção de problema
• ✨ Melhoria ou nova funcionalidade
• 📰 Nova postagem no blog

Issues relacionadas

#163

Validação

• Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
• Validei o Layout responsivo (desktop/mobile) após a implementação
• Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Anexe evidências do antes e do depois da alteração (quando necessário).

Alterações:

• Adiciona as bibliotecas @ngx-translate/core e @ngx-translate/http-loader
• Adiciona um serviço para i18n
• Altera o header para adicionar um language switcher (reaproveitado da PR I18n poc #165 )
• Adiciona um arquivo global de localization para o header/footer
• Adiciona um arquivo de localization para a home e seus componentes
• Atualiza o componente da home para exibir as strings vindas do localization file

Documentação

• A documentação deste repositório foi atualizada (quando necessário).
• Esta alteração requer que a documentação externa seja atualizada.

[netlify]

✅ Deploy Preview for querido-diario-plataforma ready!

Name	Link
🔨 Latest commit	6a267ad
🔍 Latest deploy log	https://app.netlify.com/sites/querido-diario-plataforma/deploys/673ff49f28435a000872e08d
😎 Deploy Preview	https://deploy-preview-322--querido-diario-plataforma.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[MoisesMsa]

Abri a PR porque está bem grande, como a #165 estava desatualizada a um tempo criei uma nova branch e optei por outra abordagem já que a maior parte das strings já estão mapeadas em um JSON (preciso fazer algumas melhorias quanto a isso pq algums arquivos compartilham infomações de ícones, estilos que precisei copiar pros arquivos localization para não quebrar a página).
A ideia é criar as outras PRs a partir dessa com a tradução das outras páginas e dar uma revisada se as traduções estão boas porque acho que alguns termos precisam serem melhores adptados.

[lguima]

muito bom, @MoisesMsa! excelente solução! 👏

reparei apenas que a gente traduziu a rota também, mas ela não existe em inglês, ai o link do cabeçalho fica quebrado

outro ponto é o layout mobile, foi intencional não cobrir nessa PR?

[MoisesMsa]

@lguima ajustei, as rotas de EN para as atuais em PT

[lguima]

LGTM 🚀

[ogecece]

Gente, acho que já podemos mesclar e criar as issues para as partes que faltam (mobile, outras páginas, etc.). A solução tá excelente! 1000 parabéns proceis! 1000 é pouco, aliás: https://www.youtube.com/watch?v=1Mcdh2Vf2Xk

@lguima vc poderia atualizar essa branch com rebase pra pegar a main atualizada, mesclar e criar as issues?