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

[13.0][MIG] Module l10n_br_stock_account #1784

Closed
wants to merge 377 commits into from

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Jan 19, 2022

Migration l10n_br_stock_account module #725, depends migration of modules l10n_br_account #1676 and l10n_br_stock #1776 .There are error "Unbalanced Journal Entries" when create invoice, the ideia is solve the problem in v14, this PR make the basic migrations and create the versions history because v13 in theory not will be used, will be just an passage version, if someone want use v13 should will make the backports from v14.

Migração do l10n_br_stock_account, tem um erro na criação da Fatura/Invoice de "Entradas não Balanceadas", o mesmo já relatado em outros PRs de migração da v13

image

o problema é referente ao modulo l10n_br_account na v13, portanto os testes não vão passar ( ficar verdes ) além da dependência da aprovação dos PRs de migração usados por esse modulo. Outros aspectos do modulo como a inclusão dos campos Fiscais e de Impostos na visão estão funcionando,

image

image

image

a ideia é resolver o problema na v14, esse PR faz as migrações básicas e mantém o histórico de versões, facilitando um possível "backport", porque a v13 em teoria não será usada, será apenas uma versão de passagem, se alguém quiser usar a v13 deverá fazer os "backports" da v14.

cc @renatonlima @rvalyi @marcelsavegnago @mileo

renatonlima and others added 30 commits January 17, 2022 15:11
find . -name '*.xml' ! -type d -exec bash -c 'expand -t 4 "$0" > /tmp/e && mv /tmp/e "$0"' {} \;
`procurement.move_dest_id` is not necessary to add
fiscal_position map to procurement.rule.

`partner` was already located by
openerp.addons.stock.procurement:procurement_order
mbcosta and others added 21 commits January 17, 2022 15:11
…alues was making a wrong change in the quantity.
@rvalyi rvalyi marked this pull request as draft January 19, 2022 21:30
@mbcosta
Copy link
Contributor Author

mbcosta commented Jan 20, 2022

O "commit" 2f82910 é referente a alteração do método _get_new_picking_values(), o "self" passou há conter mais de uma "stock.move", com os módulos l10n_br_sale_stock e l10n_br_purchase_stock instalados o "self" pode vir com mais de uma Operação Fiscal, nesses outros módulos é feita a alteração para usar a Operação Fiscal principal, isso é a do cabeçalho do Pedido, sem isso gera o erro abaixo:
image

@rvalyi
Copy link
Member

rvalyi commented Jun 1, 2022

Vou fechar essa PR porque nós da Akretion não temos interesse em finalizar esse PR na 13.0. já tem PR equivalente na 14.0. Pode ser reaberto se alguém se garante para terminar o trabalho na 13.0.

@rvalyi rvalyi closed this Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.