fix(po-i18n): permite mesclar contextos adicionados em "lazy modules"#421

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

rubinhos wants to merge1 commit intopo-ui:master

base:master

Choose a base branch

fromrubinhos:po-i18n/321

Open

fix(po-i18n): permite mesclar contextos adicionados em "lazy modules"#421

rubinhos wants to merge1 commit intopo-ui:masterfromrubinhos:po-i18n/321

Conversation

Copy link

Contributor

rubinhos commentedMay 21, 2020•
edited by anliben
Loading

PoI18nModule

#321

PR Checklist

Código
Testes unitários
Documentação
Samples

Qual o comportamento atual?

Atualmente quando a aplicação possui como dependência outro módulo que utiliza oPoI18nModule (uma biblioteca Angular por exemplo), as configurações de i18n da aplicação sobrescreviam as configurações de i18n do módulo importado.

Qual o novo comportamento?

Foi criado o métodoforChild que permite adicionar novos contextos para o módulo de i18n do PO. Para isto alguns ajustes foram necessários:

Agora os contextos podem ser "mesclados" (mesmo se utilizado o métodoconfig)
OproviderI18N_CONFIG agora émulti
Ao criar o serviço para o módulo atual, os contextos são mesclados
Foi adicionada uma biblioteca de terceiro que permite efetuardeep merge de objetos
Por convenção foi criado o métodoforRoot que chama o métodoconfig