l10n-brazil
l10n-brazil copied to clipboard
OCA
[14.0][REF] l10n_br_pos: Atualização nas referências para o CPF/CNPJ
Hi @lfdivino, @ygcarvalh, @luismalta, @mileo, some modules you are maintaining are being modified, check this out!
@felipezago @mileo eu tinha comentado algo sobre o nome da dependencia pos_crm que voces estão propondo https://github.com/OCA/pos/pull/836#pullrequestreview-1388544788 Talvez vale a pena botar esse PR em rascunho até solucionar essa questão da dependência, não? Outra consideração: assim como em qualquer repo da OCA/projeto open source, as pessoas vão se dar mais trabalho para revisar os PRs de vcs a medida que vcs tb estão somando para fazer trabalho naquele repo. A gente tb tem PRs 2 anos sem reviews em varios repos da OCA, é chato, mas que é assim que funciona é...
Talvez tb que pela nova dependencia seria melhor criar um novo modulo de extensão do l10n_br_pos. Isso eu não tenho certeza até que ponto essa funcionalidade é indispensavel ou opcional. O que vcs acham?
cc @renatonlima @marcelsavegnago
Essa dependência é uma das mais importantes para essa questão do PDV para a localização. Tem cenários onde é necessário ter a identificação do consumidor através do CPF/CNPJ, além de existir alguns programas do governo para quando você compartilhar essa informação durante as compras.
E criar um novo módulo só para colocar essa extensão é meio inviável. Vai deixar um ponto a mais para se ter atenção durante a manutenção, além de ter que passar muitas coisas que são a base daqui para esse outro módulo.
@rvalyi
OK então resta a ter a aprovação lá... Do meu lado se mudar o nome do pos_crm por algo mais razoável, eu me comprometo a revisar lá. Agora uma dica: não tem mais ridículo num repo OCA do que aprovação combinada de pessoa não relevante. Não sou alguém relevante no repo OCA/pos, mas eu posso revisar ainda assim e talvez ajuda a ter o módulo aceito.