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

[ADD] CTe Adapter #87

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[ADD] CTe Adapter #87

wants to merge 1 commit into from

Conversation

ODBreno
Copy link
Contributor

@ODBreno ODBreno commented Nov 10, 2023

Adicionando o CTe adapter que importa o CTe presente no PR: erpbrasil/erpbrasil.edoc#61. Vale ressaltar que estou abrindo em draft pois ainda faltam algumas mudanças que já estão prontas no PR do erpbrasil.edoc, mas que ainda não estão no repositório remoto. Assim que o CTe estiver no erpbrasil.edoc o CTe adapter deve funcionar como esperado.

@rvalyi
Copy link
Member

rvalyi commented Nov 10, 2023

Olá Breno, porém assim como no caso da MDFe em algum momento seria interessante jogar a parte de transmissão da CTe do l10n-brazil em outro PR separado. Pois esses adapter nem cheira nem fede, mas a parte do erpbrasil.edoc ela fede sim e aí fica ruim dar o merge e depois ter que depender dessas coisas para tudo... Será melhor limpar antes de dar o merge nas partes de transmissão. Mas enfim vcs conseguem usar os PRs nós projetos de vcs antes claro e se a parte de transmissão fica separada a gente consegue dar o merge dps mappings de CTe e MDFe antes claro.

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.

LGTM

@marcelsavegnago
Copy link

@rvalyi consegue tirar essa PR de rascunho ? o Breno pelo que entendi não está na Kmee

@rvalyi
Copy link
Member

rvalyi commented Jun 27, 2024

consigo ver sim

@marcelsavegnago marcelsavegnago mentioned this pull request Jul 4, 2024
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.

None yet

5 participants