search

OCA / l10n-brazil

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

[14.0][IMP] l10n_br_account: cancel invoice on fiscal document denial #3272

Open DiegoParadeda opened 1 month ago

DiegoParadeda commented 1 month ago

Adiciona cancelamento automático da fatura caso o documento fiscal seja denegado

OCA-git-bot commented 1 month ago

Hi @renatonlima, @rvalyi, some modules you are maintaining are being modified, check this out!

rvalyi commented 3 weeks ago

eu queria uma mudança sim mas comento daqui pouco.

rvalyi commented 3 weeks ago

prometo explicar essa tarde.

rvalyi commented 2 weeks ago

nao esqueci desse PR. Ainda vou fazer uns ajustes pequenos nesse PR https://github.com/OCA/l10n-brazil/pull/3012 e explicar o que teria que fazer aqui.

rvalyi commented 2 weeks ago

@DiegoParadeda entao o lance eh que tem que estancar essa sangria do modulo l10n_br_fiscal chegar na obesidade morbida... Já são mais de 20k linhas, diga-se alias muito por causa de adições do @mileo que poderiam ter sido feitas em módulos extras. Uma parte já limpamos quando extraiamos os módulos l10n_br_fiscal_dfe, l10n_br_fiscal_closing e l10n_br_fiscal_certificate no inicio da v14. Porem ta na hora de extrair ainda esse l10n_br_fiscal_edi pelo menos na v16. Ai vem a questão se não é melhor encarar isso na v14 ainda para não matar a sinergia entre a v14 e as versões mais novas. Sou dos que pensam que deixar a cagada para limpar depois vai custar ainda 10x mais do que limpar quanto antes...

Nisso a ideia seria proceder com o merge de #3012 primeiro. E depois fazer um override desse novo hook: exec_after_SITUACAO_EDOCDENEGADA (sem ) https://github.com/OCA/l10n-brazil/pull/3012/files#diff-f146a34ad7da33eb19b281e63a3d4dbe42724fb3c3e8c9eaa107c537f280b4a2R445 Assim evita que a localização vira um grande cocô onde tudo depende de tudo e ninguém não controla mais nada.

Pode ser debatido quais hooks teria ou não porem, o urgente eh estruturar melhor a localização e extrair este modulo l10n_br_fiscal_edi antes de impactar mais gente e mais módulos com essa mudança...

cc @renatonlima @mbcosta @marcelsavegnago @antoniospneto @felipemotter @kaynnan @douglascstd

rvalyi commented 6 days ago

@DiegoParadeda fizemos o merge de #3012 para não perder a sinergia com a v16, vc consegue dar um rebase e fazer o que eu sugeri no comentário antes? cc @mileo

antoniospneto commented 23 hours ago

@DiegoParadeda o que acha de fazer um pequeno teste unitário para esse caso?

marcelsavegnago commented 13 hours ago

@DiegoParadeda o que acha de fazer um pequeno teste unitário para esse caso?

boa... bora evitar regressões.