[14.0][IMP] add delivery_nfe volumes

[DiegoParadeda]

Adiciona volumes (nfe40_vol) gerados automaticamente via:

1. Wizard de faturamento no stock.picking
2. Wizard de geração de volumes no stock.picking

[OCA-git-bot]

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

[rvalyi]

Ta tranquilo vc prototipar ou ate atender clientes com strings dos campos em português. Porem hoje, na OCA, a gente ja tem que pensar em empresas multi-pais usando e nisso é bom uar strings em ingles (apenas se for diferente do nome da variável) e traduzir por cima. Não avaliei a questão da lógica das coisas que vc ta propondo aqui.

[DiegoParadeda]

Ta tranquilo vc prototipar ou ate atender clientes com strings dos campos em português. Porem hoje, na OCA, a gente ja tem que pensar em empresas multi-pais usando e nisso é bom uar strings em ingles (apenas se for diferente do nome da variável) e traduzir por cima. Não avaliei a questão da lógica das coisas que vc ta propondo aqui.

resolvido!

[DiegoParadeda]

@mbcosta acho que consegui corrigir todos os pontos que você levantou, obrigado pelo review!

[mileo]

@DiegoParadeda pode fazer um rebase e squash?

[rvalyi]

ai tb pode se adicionar nos contribuidores do módulo porque é uma contribuição significativa (apesar disso eu não revisei direito ainda).

[rvalyi]

ai tb pode se adicionar nos contribuidores do módulo porque é uma contribuição significativa (apesar disso eu não revisei direito ainda).

na real, o modulo era muito pequeno e muito simples e vcs fizeram até mais codigo do que a gente nele... Nisso eu diria que pode até adicionar a KMEE de co-autor. Novamente não revisei, isso é assumindo que ta legal...

cc @renatonlima @mbcosta @marcelsavegnago

[mileo]

1. Já que estamos colocando dependências, poderia incluir o product_brand? https://github.com/OCA/brand/tree/16.0/product_brand
2. Não faz sentido ter os campos em ambos os modelos dos produtos, template e variante. Coloque somente onde esta o peso (se lembro de cabeça fica na variante);

[mileo]

@DiegoParadeda melhor colocar os campos na variante, como https://github.com/odoo/odoo/blob/14.0/addons/product/models/product.py#L118

A marca se formos usar a dependência, via ficar no template, mas isso pode ser alterado por um terceiro módulo.

[rvalyi]

eu diria que pode depender do product_brand. Novamente é um modulo muito utilizado e trivial. Sendo que o proprio l10n_br_delivery_nfe é um modulo muito pequeno tb.

[marcelsavegnago]

eu diria que pode depender do product_brand. Novamente é um modulo muito utilizado e trivial. Sendo que o proprio l10n_br_delivery_nfe é um modulo muito pequeno tb.

Concordo também com esta dependencia.

[mileo]

@DiegoParadeda pode incluir nas dependências a integração com o módulo: https://github.com/OCA/delivery-carrier/tree/14.0/delivery_package_number

Da forma como esta modelado a informação do nº de volumes no picking fica inexistente, vejo duas opções:

1 - Criar um campo calculado que leia as informações totais do nº de volumes do documento fiscal;
2 - Armazenar a informação do total de volumes no campo do módulo citado acima.

[mileo]

@DiegoParadeda pode incluir nas dependências a integração com o módulo: https://github.com/OCA/delivery-carrier/tree/14.0/delivery_package_number

Da forma como esta modelado a informação do nº de volumes no picking fica inexistente, vejo duas opções:

1 - Criar um campo calculado que leia as informações totais do nº de volumes do documento fiscal; 2 - Armazenar a informação do total de volumes no campo do módulo citado acima.

Entretanto esse módulo coloca um conjunto de wizards:

https://github.com/OCA/delivery-carrier/tree/14.0/delivery_package_number/wizard

Em que a integração com ele pode não ficar boa e seja exigido um modelo concreto relacionado ao picking.

Corrigido

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[rvalyi]

considerando os fixes recentes e as aprovações...

/ocabot merge major

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3091-by-rvalyi-bump-major, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 303369f. Thanks a lot for contributing to OCA. ❤️