Skip to content

Conversation

@rvalyi
Copy link
Member

@rvalyi rvalyi commented Oct 12, 2025

Isso permite que a lib erpbrasil.edoc seja usada com versões recentes da nfelib depois dessa mudança no ano passado: akretion/nfelib#106

EDIT: isso é apenas uma copia da antiga pasta dos bindings de NFe gerido pelo generateDS que o erpbrasil.edoc ainda usa da nfelib 2.0.7. O erpbrasil.edoc precisa apenas dos bindings de transmissão mas infelizmente o generateDS não sabia gerar arquivos de forma modular como o xsdata faz entaõ esses bindings são misturado no mesmo arquivo gigantão de 25k linhas com os bindings do leiaute de NFe. Mas enfim a ideia é que o erpbrasil.edoc usa esses bindings de transmissão generateDS legacy mas que pro leiaute da NFe vc consiga usar os bindings mais recente das versões recentes da nfelib que atendem inclusive a reforma tributaria...

@rvalyi
Copy link
Member Author

rvalyi commented Nov 5, 2025

pessoal, aqui os testes não passaram, mas eles tb não passaram nos outros PRs recentes que entraram; não parece relacionado as minhas mudanças aqui...

Por outro lado eu fiz um PR no OCA/l10n-brazil que ficou verde com varios testes que testam um pouco o uso do erpbrasil.edoc OCA/l10n-brazil#4214

@rvalyi rvalyi marked this pull request as ready for review November 5, 2025 20:40
Copy link
Member

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

Por mim tá OK, testei apenas as questões relacionadas a NFe.

@rvalyi
Copy link
Member Author

rvalyi commented Nov 6, 2025

OBS: para quem usa o erpbrasil.edoc na versão 12.0 ainda por exemplo isso não ira bem permitir de usar versões novas da nfelib (porque versões recentes do xsdata não rodam no Python 3.7 e porque a branch 12.0 OCA/l10n-brazil ainda usa versoes da nfelib com generateDS). Mas tambem essa mudança não deve impedir de usar o erpbrasil.edoc com essas versões antigas da nfelib, pois apenas vai usar uma copia dos bindings dos eventos dessa nova pasta vendorizada. Na 12.0 isso não vai resolver a questão da reforma tributaria. Na 14.0 porem deve resolver a questão da reforma tributaria com backport de OCA/l10n-brazil#4214 e backports de outras mudanças ligadas a reforma tributaria.

@mileo mileo merged commit 0723b34 into erpbrasil:master Nov 8, 2025
2 of 11 checks passed
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