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

[14.0][IMP] l10n_br_mdfe: add transmission, cancelling and closure #3526

Open
wants to merge 8 commits into
base: 14.0
Choose a base branch
from

Conversation

marcelsavegnago
Copy link
Member

No description provided.

@OCA-git-bot
Copy link
Contributor

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

@mileo
Copy link
Member

mileo commented Dec 5, 2024

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

@marcelsavegnago
Copy link
Member Author

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

Acredito que hoje quando o l10n_br_account esta instalado, ao criar o documento fiscal o account move ẽ criado junto.

@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch 2 times, most recently from 0496f83 to 18509ae Compare December 5, 2024 14:39
@rvalyi
Copy link
Member

rvalyi commented Dec 5, 2024

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

Acredito que hoje quando o l10n_br_account esta instalado, ao criar o documento fiscal o account move ẽ criado junto.

Nope, cria apenas o documento fiscal no caso. Eh apenas o contrário que acontece se criar um account.move com um fiscal_operation_id ai cria documento fiscal junto.

no final do account_move.py tem uma função para importar account.move a partir de um documento fiscal, mas é um API que tem que ser chamado propositalmente (por examplo na importação de uma NFe).

do restante pode ser que o @mileo tenha razão e que nao devemos criar account.move para a MDFe...

@antoniospneto
Copy link
Contributor

Sobre a questão de atrelar a fatura ao MDFe, se não há nenhuma complicação técnica, concordo que é melhor não criar a fatura.

@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented Dec 5, 2024

Sobre a questão de atrelar a fatura ao MDFe, se não há nenhuma complicação técnica, concordo que é melhor não criar a fatura.

Boa., eu tbm acho que eh bem melhor mesmo.. Enfim, vamos tratar isso em outra PR e deixar esta aqui em paralelo ja que ela soh trata a transmissao mesmo..ai depois faco rebase

@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch 2 times, most recently from d1a20b8 to 64f2cd9 Compare December 10, 2024 13:12
@rvalyi
Copy link
Member

rvalyi commented Dec 10, 2024

parem ele! chega de PR de novo documento fiscal tudo verdinho! parece até que vc gosta desses documentos fiscais.

@marcelsavegnago
Copy link
Member Author

agora sim :D tudo verdim

@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch from 64f2cd9 to 91bf909 Compare December 10, 2024 13:31
@marcelsavegnago
Copy link
Member Author

Pronto.. soh mais um push para acertar o commit :D

@marcelsavegnago marcelsavegnago marked this pull request as ready for review December 10, 2024 16:43
@marcelsavegnago
Copy link
Member Author

@mileo @antoniospneto @rvalyi sobre a criacao de documento fiscal sem o account.move, fiz alguns testes e de fato da para fazer... hoje do jeito que esta ja eh "possivel".. Porem, precisamos de alguns ajustes para evitar entrada incorreta dependendo do local que o usuario acessar. Sendo assim, nesta PR vou abstrair disso ja que o esforco maior sera mesmo em trabalhar no modulo l10n_br_account e no l10n_br_fiscal para viabilizar. Fiz uma PR de esboco #3536

@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch 2 times, most recently from 2eb6b11 to 1400473 Compare December 10, 2024 17:22
@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch 4 times, most recently from d71c694 to 362cc04 Compare December 10, 2024 22:21
@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch from 362cc04 to dbd9bcb Compare December 11, 2024 12:20
@marcelsavegnago marcelsavegnago force-pushed the 14.0-imp-l10n_br_mdfe-transmission branch from dbd9bcb to fe762f4 Compare December 11, 2024 12:50
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.

5 participants