OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
241 stars 245 forks source link

[RFC][10.0][12.0] Remover a dependência do módulo l10n_br_account_product no l10n_br_account_payment_mode #671

Closed renatonlima closed 4 years ago

renatonlima commented 5 years ago

Como conversado no PR #655,

Existe um erro de dependência no módulo l10n_br_account_payment_mode que dependente do módulo l10n_br_account_product, na verdade o módulo l10n_br_account_payment_mode deveria depender apenas do l10n_br_account

Para corrigir essa dependência seria necessário mover dois campos em: https://github.com/akretion/l10n-brazil/blob/8.0/l10n_br_account_payment_mode/models/payment_mode.py#L12

type_sale_payment e type_payment para o módulo l10n_br_account_product, inclusive esses campos estão duplicados, só deveria existir o campo type_payment

Também deveria ser refatorado a estrutura dos módulos de cobrança e boletos pois existem outros problemas de modularidades que deveriam ser resolvidos, mas eu acredito que no processo de migração na branch 10.0 devemos deixar como estar e deixar para fazer a mudança de depedencia junto com o refatoramento da modularidade de todos os módulos de cobrança e boletos na versão 12.0

cc @mileo @hendrixcosta @rvalyi @mbcosta

mbcosta commented 5 years ago

Ola @renatonlima , como a alteração para remover a dependência do l10n_br_account_product já está sendo feita no no PR https://github.com/OCA/l10n-brazil/pull/695 assim como a reestrutura do modulos sugiro alterar a "tag" 12.0 para 10.0