Skip to content

Conversation

@rvalyi
Copy link
Member

@rvalyi rvalyi commented Dec 10, 2025

Na hora do merge de #4072 teve uma regressão: o método _compute_fiscal_amount não esta mais sendo herdado e precisa ser acessado através do fiscal_document_id

@rvalyi rvalyi marked this pull request as draft December 10, 2025 03:54
@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago
Copy link
Member

Opa.. fica em rascunho ainda ?

@antoniospneto
Copy link
Contributor

@rvalyi mas será que ainda precisamos chamar esse compute explicitamente? consegue compartilhar onde que ocorre essa regressão?

Se for possivel, eu penso que seria melhor deixar o api.depends decidir quando o compute deve ser chamado.

de qualquer forma se essa é a solução para destravar algo dá pra dar sequencia assim..

@rvalyi
Copy link
Member Author

rvalyi commented Dec 11, 2025

@rvalyi mas será que ainda precisamos chamar esse compute explicitamente? consegue compartilhar onde que ocorre essa regressão?

Se for possivel, eu penso que seria melhor deixar o api.depends decidir quando o compute deve ser chamado.

de qualquer forma se essa é a solução para destravar algo dá pra dar sequencia assim..

A gente precisou num código de importação de NFe custom que migramos e que não é o codigo do repo. sem isso os moves ficam desbalanceados. mas eu tava querendo verificar tb com o teste de importação do move composto que vc comentou onde eu acho que fazia diferença. Vou ver isso em breve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants