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

[16.0][WIP][REF] l10n_br_account_due_list -> account_move_installment #3362

Draft
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Sep 13, 2024

WIP - A ideia é jogar pro OCA/account-invoicing depois. Até onde eu conferi, até na v18 um modulo genérico assim vai fazer sentido. Nas v17 e v18 o Odoo passou a falar de "installment" no modulo account, porem esse tipo de aba pode realmente fazer sentido nos países on se paga com muitas parcelas e onde tem bastante lançamentos de taxas, o que torna a visualização das parcelas na aba dos lançamentos nativa de difícil uso.

Também não precisa depender do modulo account_due_list na altura deste modulo. Basta depender no l10n_br_account_payment_order (ja é o caso) e sugerir o uso no README do l10n_br_account.

Tou tb renomeando os campos para receivable_line_ids e payable_line_ids. Depois de tudo verde, irei submeter o modulo OCA/account-invoicing na v16 ou na v17 e iremos ver o que fazemos aqui na v16 (mas seria mole tirar daqui se o modulo for aceito no OCA/account-invoicing na v16).

There are only two hard things in Computer Science: cache invalidation and naming things.

-- Phil Karlton

aquela coisa...

(na real, o @renatonlima tinha extraído o l10n_br_account_due_list as pressas na v12 pro modulo l10n_br_account_payment_order não depender do l10n_br_account e tinha essa margem para melhorar)

Nota: como esse módulo é minimalista e super maduro, não vejo necessidade de fazer backport disso para v14 ou v15.

@rvalyi rvalyi marked this pull request as draft September 13, 2024 00:19
@antoniospneto
Copy link
Contributor

Acho que já tem gente fora da localização usando, pois foi feito tradução para o espanhol.

Vai ser uma boa mesmo

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.

2 participants