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

[MIG] l10n_br_account: Number in payment terms invoice lines #220

Conversation

antoniospneto
Copy link

@antoniospneto antoniospneto commented Jul 27, 2024

Proposta para migração da funcionalidade que adiciona o número da parcela no contas a receber.

image

Fiz um override na função de contexto _sync_dynamic_lines, mas uma outra opção viavel é sobreescever os métodos write() e create()

O que acha @rvalyi ?

@github-grap-bot
Copy link

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

@antoniospneto
Copy link
Author

esse erro que tá dando é só quando o módulo l10n_br_account_payment_order tá instalado, vou ver o que é

@rvalyi
Copy link
Member

rvalyi commented Jul 27, 2024

Valeu Antônio, já vou ver. Eu tive que desativar uns testes no payment order, se vc conseguir ajudar nisso...

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_account branch 2 times, most recently from 82c859a to f099b4a Compare July 28, 2024 00:33
@rvalyi
Copy link
Member

rvalyi commented Jul 28, 2024

valeu @antoniospneto eu integrei assim por hora. Fiz um cherry-pick, e alterei o commit de antes para reduzir o diff (deixei _recompute_payment_terms_lines em vez de TODO_recompute_payment_terms_lines) e remanejei seu commit no meld.

Enfim contudo vc pode ver aqui onde foi integrado:
OCA@3244dc8

Ficou perfeito colaborar dessa forma onde vc resolve uma pequena parte des coisas nao migrada e vou integrando assim.

cc @renatonlima @marcelsavegnago

@rvalyi
Copy link
Member

rvalyi commented Jul 28, 2024

Nisso vou fechar o PR ja que integrei com cherry-pick

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.

3 participants