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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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

wants to merge 1 commit into from

Conversation

rubinhos
Copy link
Contributor

@rubinhos rubinhos commented May 21, 2020

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 o PoI18nModule (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étodo forChild 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étodo config)
  • O provider I18N_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 efetuar deep merge de objetos
  • Por convenção foi criado o método forRoot que chama o método config
  • Documentação e testes alterados.

Simulação

Executar a simulação relatada na ISSUE #321.

Sorry, something went wrong.

@fabiomr78 fabiomr78 added bug Something isn't working and removed bug Something isn't working labels Jun 16, 2020
@alvarocamillont alvarocamillont added the hacktoberfest-accepted Aceito para a hacktoberfest label Oct 27, 2020
@jhonyeduardo jhonyeduardo removed the hacktoberfest-accepted Aceito para a hacktoberfest label Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants