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

[15.0][MIG] l10n_br_sale_stock #3531

Draft
wants to merge 241 commits into
base: 15.0
Choose a base branch
from

Conversation

antoniospneto
Copy link
Contributor

Rebaertura do #3368

rvalyi and others added 30 commits December 3, 2024 22:23
…l10n_br_sale_stock module to accomodate to the increased modulrity in OpenERP 7 where sale modules doesn't force you to install the stock module aymore
…br_crm, l10n_br_data_base, l10n_br_data_zip e l10n_br_sale_stock
…stock e corrigido métodos onchange do objeto stock.picking
…emo nos arquivos __openerp__.py de todos modulos da localização
…sale_stock, removido chave 'demo' duplicada no l10n_br_sale_stock/__openerp__.py
… fiscais e copia-la nos documentos fiscais, corrigido copia de observações de posições fiscais por linhas
…ento fiscal a partir do picking, postado códigos para manter modularidade.
… de alertas no log e futura internacionalização dos termos da localização
…module, preserving all orginial on_change signatures
…br_sale and works wether l10n_br_account_service is installed or not
…t so it can be shared bewteen products and services. You need to change the ir_model_data id of this fiscal category by SQL to update your database
…r_account so it can be shared bewteen products and services. You need to change the ir_model_data id of this fiscal category by SQL to update your database"

This reverts commit 5296355.
OCA-git-bot and others added 22 commits December 3, 2024 22:23
Currently translated at 47.0% (8 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 52.9% (9 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 52.9% (9 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 58.8% (10 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 64.7% (11 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 70.5% (12 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
Currently translated at 100.0% (17 of 17 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_sale_stock
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_sale_stock/pt_BR/
When Picking has a Sale Order related the Partner used to create the Invoice should be the partner_invoice_id of Sale, because the Partner of Picking can be the partner_shipping_id of Sale Order.

[FIX] l10n_br_sale_stock: Get Fiscal Partner
When mapping the Line Fiscal Operation and Taxes the Partner of the object can be or not the Partner to Invoice, in case of Picking with related a related SO, it should use the partner_invoice_id field in Sale because the Partner of Picking can be the partner_shipping_id of the SO.
@antoniospneto
Copy link
Contributor Author

/ocabot migration l10n_br_sale_stock

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Dec 7, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 7, 2024
30 tasks
Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seria interessante levar em consideração #2955 e OCA/account-invoicing#1826

@mbcosta
Copy link
Contributor

mbcosta commented Dec 18, 2024

Por favor, solicito ao @antoniospneto e aos outros PSC do projeto @OCA/local-brazil-maintainers considerar o Bloqueio ou mesmo o Fechamento desse PR com o objetivo de forçar a Revisão do PR [14.0][REF] l10n_br_sale_stock: Extraction referent the creation of the module sale_stock_picking_invoicing os motivos são:

  • O PR torna desnecessário a migração do módulo [14.0][MIG] l10n_br_sale_commission_stock #2681 que apesar de ser simples representa os vários possíveis casos de necessidade de criar "glue modules" na implementação que existe hoje na v14 e nessa migração, podem existir diversos casos por exemplo account_payment_sale ou sale_commission, quer dizer se for feito o merge da forma que está a Localização ainda vai "precisar" ou tem a responsabilidade de migrar o l10n_br_sale_commission_stock na v14, v15 e v16 para ter isso funcionando e ainda resolver caso a caso os diversos campos específicos que podem não estar sendo copiados

  • A melhoria feita 'é considerável, pode ser demonstrada é comprovada através dos Testes do código e na tela com Dados de Demonstração que permitem ver e testar os casos de uso sem necessidade de muitas parametrizações

Isso não é um trabalho de dias, semanas ou meses são anos que estamos buscando fazer essa Extração e melhorar essa implementação, houveram PRs e Issues questionando o módulo porque acabavam faltando campos quando a criação ocorria através do Picking, eu tenho buscado orientar e resolver a maior parte dos Issues referentes a implementação de Criação de Faturas através do Picking e também criando tanto os PRs de Migração, Melhorias, Manutenções e Correções; diante disso peço que seja avaliada essa questão simplesmente para evitar gastar tempo em uma implementação que está sendo melhorada de forma efetiva.

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.