Skip to content

Conversation

@rvalyi
Copy link
Member

@rvalyi rvalyi commented Nov 5, 2025

depende de erpbrasil/erpbrasil.edoc#92

Esse PR visa a demonstrar que é viável fazer o merge desse PR no erpbrasil.edoc erpbrasil/erpbrasil.edoc#92 para poder usar versões mais recentes de nfelib.

Isso seria um primeiro passo para:

  1. trabalhar com a nfelib para a reforma fiscal (IS/IBSCBS...)
  2. se livrar da dependência erpbrasil.edoc no modulo l10n_br_fiscal_dfe com [16.0][REF][IMP] l10n_br_fiscal_dfe, l10n_br_nfe: Melhorias e Implementações na distribuição do DFe #4096
  3. se livrar da dependência erpbrasil.edoc no modulo l10n_br_nfe com [16.0] use experimental nfelib version with SOAP support #4147

OBS: como eu tive o cuidado de testar a nfelib com Python 3.8, seria viavel fazer o backport disso para a branch 14.0 tb, para suportar a reforma tributaria por exemplo...

@OCA-git-bot
Copy link
Contributor

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

@rvalyi rvalyi marked this pull request as draft November 5, 2025 19:25
@antoniospneto
Copy link
Contributor

Testei aqui em ambiente de homologação com uma empresa de SC:

Transmissão de NFe: OK
Cancelamento: OK
Carta de Correção: OK
Consultar Documento: OK
Inutilizar numeração: OK

Testei apenas questões relacionadas a NFe.

@antoniospneto
Copy link
Contributor

erpbrasil/erpbrasil.edoc#92

Mesclado :D

@rvalyi
Copy link
Member Author

rvalyi commented Nov 8, 2025

teria que ter um release para a gente poder passar esse como ready. Sendo que provavelmente agora o problema observado com erpbrasil edoc deve travar dos os PRs eu imagino...

@antoniospneto
Copy link
Contributor

to gerando o realease aqui

@antoniospneto
Copy link
Contributor

antoniospneto commented Nov 8, 2025

Comment on lines 151 to 158
with (
mock.patch.object(
DocumentoElectronicoAdapter,
"_post",
side_effect=mocked_post_confirmacao_status_code_error,
),
self.assertRaises(ValidationError),
):
Copy link
Contributor

Choose a reason for hiding this comment

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

esse diff é necessario?

Copy link
Member Author

Choose a reason for hiding this comment

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

não, foi que veio qdo usei o ruff, mas posso tirar

@rvalyi rvalyi force-pushed the 16.0-nfelib-master branch from 88a7ce0 to 8aeaff9 Compare November 8, 2025 17:32
@rvalyi
Copy link
Member Author

rvalyi commented Nov 8, 2025

@antoniospneto fiz rebase e agora temos a nova treta do erpbrasil.base:

2025-11-08 17:35:47,680 501 WARNING odoo odoo.modules.module: DistributionNotFound: The 'erpbrasil.base>=2.3.0' distribution was not found and is required by the application 
2025-11-08 17:35:47,681 501 WARNING odoo odoo.modules.loading: Transient module states were reset 
2025-11-08 17:35:47,682 501 ERROR odoo odoo.modules.registry: Failed to load registry 
Traceback (most recent call last):
  File "/opt/odoo/odoo/modules/module.py", line 538, in check_python_external_dependency
    pkg_resources.get_distribution(pydep)
  File "/opt/odoo-venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 471, in get_distribution
    dist = get_provider(dist)
  File "/opt/odoo-venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 347, in get_provider
    return working_set.find(moduleOrReq) or require(str(moduleOrReq))[0]
  File "/opt/odoo-venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 891, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/opt/odoo-venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 777, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'erpbrasil.base>=2.3.0' distribution was not found and is required by the application

@rvalyi
Copy link
Member Author

rvalyi commented Nov 9, 2025

pessoal erpbrasil/erpbrasil.edoc#92 já foi mesclado e agora as mudanças desse PR estão incluída no PR #4223 que também resolve outros problemas que apareceram com as versões das libs erpbrasil. Vamos fechar esse pois vamos mesclar #4223.

@rvalyi rvalyi closed this Nov 9, 2025
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.

3 participants