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][FIX+IMP] l10n_br_sale_stock: Inform Fields that should not be used from Sale 'prepare' methods, tests, FWD 3619 #3636

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Feb 20, 2025

Foward Port #3619

Inform Fields that should not be used from Sale 'prepare' methods, depends OCA/account-invoicing#1919

Permite informar campos que vêm dos métodos "prepare" do Sale e do Sale Line mas que não devem ser usados; no caso os campos Número do Documento e Série devido ao caso de uso onde os valores já são informados dentro do Picking.

Seria importante uma ajuda no repo do account-invoicing na Revisão de PRs relacionados ao uso aqui na Localização existe tanto esse PR que na v16 como outros que agradeço se a comunidade puder ajudar

FWD PR similar porém na v15
https://github.com/OCA/account-invoicing/pull/1918

FWD Caso DropShipping
Algo simples mas necessário para carregar corretamente o Tipo da Invoice no caso de uso DropShipping
v15
https://github.com/OCA/account-invoicing/pull/1891
v16
https://github.com/OCA/account-invoicing/pull/1892

Coloquei dentro do "bash" para não marcar diretamente os PRs por não terem uma relação direta com esse PR , mas como escrevi são melhorias em módulos usados aqui.

cc @OCA/local-brazil-maintainers

@OCA-git-bot
Copy link
Contributor

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

@mbcosta mbcosta force-pushed the 16.0-FIX-sale_stock_rm_fields_from_sale branch from d083c3d to a2d6d06 Compare February 20, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants