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][ADD] l10n_br_cte: add new module #3492

Open
wants to merge 17 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, @rvalyi,
some modules you are maintaining are being modified, check this out!

@marcelsavegnago marcelsavegnago force-pushed the 14.0-add-l10n_br_cte-rebased-multi_schema-refactor branch from e9383db to 778ea20 Compare November 14, 2024 17:55
@marcelsavegnago marcelsavegnago marked this pull request as ready for review November 14, 2024 17:58
@marcelsavegnago
Copy link
Member Author

Ainda vou ajustar os commits e deixar mais organizado.

@marcelsavegnago
Copy link
Member Author

@mileo da uma olhada no arquivo de contribuintes e se tiver mais alguem me passa o nome e o email aqui por favor.

@marcelsavegnago
Copy link
Member Author

@mileo por favor peça para seu time dar uma revisada.. por ora comentei os trechos da transmissao mas ja da para validar a geracao do xml.

@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented Nov 14, 2024

@rvalyi da uma olhada neste commit por favor e veja se faz sentido algo assim para voce.. df937a4

@rvalyi
Copy link
Member

rvalyi commented Nov 14, 2024

@rvalyi da uma olhada neste commit por favor e veja se faz sentido algo assim para voce.. df937a4

Então teve essa atualização importante #3127

Vale a pena testar se continua funcionando com essa atualização... talvez dar o merge dela primeiro ou pelo menos conferir num PR...

@marcelsavegnago
Copy link
Member Author

@rvalyi da uma olhada neste commit por favor e veja se faz sentido algo assim para voce.. df937a4

Então teve essa atualização importante #3127

Vale a pena testar se continua funcionando com essa atualização... talvez dar o merge dela primeiro ou pelo menos conferir num PR...

vou dar mais uma tapa nas coisas para ter algo rodando bonitinho e ja testo com esta PR..

Usage
=====

Para utilizar o módulo `l10n_br_cte` em conjunto com o módulo `l10n_br_account`, é necessário configurar uma linha de operação fiscal que não adicione valor ao montante do documento, uma vez que o CT-e (Manifesto Eletrônico de Documentos Fiscais) não possui valor financeiro.
Copy link
Contributor

Choose a reason for hiding this comment

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

acho que aqui misturou o conceito de ct-e com mdf-e não?

Copy link
Member Author

Choose a reason for hiding this comment

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

foi o control-c, control-v :D .. vlwwww vou ajustar

Comment on lines +75 to +76
Seguindo esses passos, o módulo `l10n_br_cte` funcionará corretamente em conjunto com o `l10n_br_account`, permitindo a emissão de CT-e sem valores financeiros associados.

Copy link
Contributor

Choose a reason for hiding this comment

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

aqui a mesma duvida, cte pode ter valor, é o mdfe que não, ou estou errado?

Copy link
Member Author

Choose a reason for hiding this comment

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

exatamente

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.

2 problemas nos meus testes superficiais:

1: faltou o menu de CTe:
2024-11-14_19-45

2: erro quando usa o menu "all documents" e depois create:
2024-11-14_19-43

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