[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081

antoniospneto · 2024-05-15T14:38:19Z

Continuação dos trabalhos feitos nas PRs #2945 e #3065.

As duas PRs são bastante conflitantes, por isso optei por juntar o trabalho aqui.

Resumo das melhorias propostas nesta PR:

Adição do suporte para transmissão síncrona.
Melhoria no processo de envio assíncrono para evitar perda de dados em caso de falha.
Implementação do wizard de consulta de situação para a NF-e.
Suporte para transmissão normal, utilizando as Sefaz autorizadoras de contingência.
Outras pequenas correções de usabilidade.

Depende de:

Falta fazer:

Opção de habilitar a sefaz de contingencia nos outros serviços: Inutilização, Cancelamento, Carta de Correção.
Implementar mais testes

OCA-git-bot · 2024-05-15T14:38:25Z

Hi @luismalta, @mileo, @gabrielcardoso21, @renatonlima, @ygcarvalh, @rvalyi, @AndreMarcos, @marcelsavegnago,
some modules you are maintaining are being modified, check this out!

rvalyi · 2024-05-15T20:10:23Z

fiz o merge de akretion/nfelib#103
e fiz o push da nfelib-2.0.6 no pypi

Não sei se isso permite aos testes de rodar, vou mandar rodar...

l10n_br_nfe/tests/test_nfe_webservices.py

- Added support for contingency mode (SVC) - Added support for synchronous transmission - Added NFe status check functionality - Improved transmission process to ensure information received from SEFAZ is not lost on failure - Various performance optimizations and bug fixes nfe

…into 14.0-merged-features2 OCA#3081

rvalyi

LGTM.

@mileo vcs que sao co-autores do modulo e fizeram a maior parte do API de transmissão, alguma posiçao sobre esse refator importante?

cc @renatonlima @marcelsavegnago

renatonlima · 2024-06-12T16:37:45Z

@antoniospneto,

Muito boa as contribuições, há um tempo atrás eu fiz um protótipo para refatorar todo o código do l10n_br_fiscal.document, l10n_br_fiscal.document.eletronic, l10n_br_fiscal.document.workflow, l10n_br_fiscal.event no módulo l10n_br_fiscal, eu acredito que dá para limpar bastante código e estruturar melhor os objetos e métodos porque há muito código no l10n_br_fiscal e deveria estar em outros módulos como l10n_br_nfe, l10n_br_cte, l10n_br_nfse.

Eu diria que seria ótimo a gente fazer esse PR e depois portar para a 15.0 e 16.0 com isso eu iria conseguir integrar com o trabalho de refatoramento que estou fazendo.

rvalyi · 2024-06-12T16:43:21Z

@renatonlima @antoniospneto @marcelsavegnago depois deste merge e antes de adicionar mais funcionalidades na gestao dos eventos seria interessante dar uma limpa e extrair este modulo l10n_br_fiscal_edi em fez de aumentar a obesidade morbida do modulo l10n_br_fiscal porem #3012 Afinal de conta essa extracao parece bem tranquila.

mileo · 2024-06-12T19:31:42Z

@antoniospneto,

Muito boa as contribuições, há um tempo atrás eu fiz um protótipo para refatorar todo o código do l10n_br_fiscal.document, l10n_br_fiscal.document.eletronic, l10n_br_fiscal.document.workflow, l10n_br_fiscal.event no módulo l10n_br_fiscal, eu acredito que dá para limpar bastante código e estruturar melhor os objetos e métodos porque há muito código no l10n_br_fiscal e deveria estar em outros módulos como l10n_br_nfe, l10n_br_cte, l10n_br_nfse.

Eu diria que seria ótimo a gente fazer esse PR e depois portar para a 15.0 e 16.0 com isso eu iria conseguir integrar com o trabalho de refatoramento que estou fazendo.

Vou dar uma testada com NFC-e ainda essa semana e dou um retorno.

rvalyi · 2024-06-12T20:08:13Z

@antoniospneto,
Muito boa as contribuições, há um tempo atrás eu fiz um protótipo para refatorar todo o código do l10n_br_fiscal.document, l10n_br_fiscal.document.eletronic, l10n_br_fiscal.document.workflow, l10n_br_fiscal.event no módulo l10n_br_fiscal, eu acredito que dá para limpar bastante código e estruturar melhor os objetos e métodos porque há muito código no l10n_br_fiscal e deveria estar em outros módulos como l10n_br_nfe, l10n_br_cte, l10n_br_nfse.
Eu diria que seria ótimo a gente fazer esse PR e depois portar para a 15.0 e 16.0 com isso eu iria conseguir integrar com o trabalho de refatoramento que estou fazendo.

Vou dar uma testada com NFC-e ainda essa semana e dou um retorno.

Na real, mesmo que tem coisa de NFCe no diff ele nao alterou nehnuma logica da NFCe, mas ainda assim como eh um refator grande e que mexe nas partes que voces fizeram eh interessante vc confirmar.

rvalyi · 2024-06-16T22:39:08Z

@mileo conseguiu ver se tinha algum problema para vcs neste PR?

l10n_br_nfe/models/document.py

mileo · 2024-06-20T13:39:35Z

@mileo conseguiu ver se tinha algum problema para vcs neste PR?

Estamos validando com um cliente esse pr hj.

OCA-git-bot · 2024-06-28T21:00:02Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

rvalyi · 2024-07-04T18:30:45Z

@mileo pedi seu retorno porque são vcs que fizeram esse codigo da transmissão da NFe sendo refatorado aqui. Mas tb vejo que todos estamos achando o refator top, vc pretende testar ainda ou podemos fazer o merge?

Veja que uma coisa é botar coisas novas como recuperação de crédito, outra coisa é botar mais qualidade no codigo legacy que ainda temos... Essa segunda coisa tem que ser muito mais prioritária antes de espelhar código mais ou menos em branches diferentes onde cada um vai depois querer refatorar de um jeito diferente para atender X ou Y...

mileo · 2024-07-05T01:18:40Z

@mileo pedi seu retorno porque são vcs que fizeram esse codigo da transmissão da NFe sendo refatorado aqui. Mas tb vejo que todos estamos achando o refator top, vc pretende testar ainda ou podemos fazer o merge?

Veja que uma coisa é botar coisas novas como recuperação de crédito, outra coisa é botar mais qualidade no codigo legacy que ainda temos... Essa segunda coisa tem que ser muito mais prioritária antes de espelhar código mais ou menos em branches diferentes onde cada um vai depois querer refatorar de um jeito diferente para atender X ou Y...

Boa noite, ainda estamos revisando, tem uma pessoa do time validando o problema da NFC-e e estamos testando esse PR em um cliente desde a semana passada, como tem bastante código e depois de mergado fica complicado trabalhar sem ele.

mileo · 2024-07-10T21:19:04Z

/ocabot merge major

OCA-git-bot · 2024-07-10T21:19:19Z

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3081-by-mileo-bump-major, awaiting test results.

OCA-git-bot · 2024-07-10T21:51:44Z

Congratulations, your PR was merged at 3af68b3. Thanks a lot for contributing to OCA. ❤️

This was referenced May 15, 2024

[14.0] Melhorias na transmissão da NFE #2945

Closed

[14.0][IMP] l10n_br_nfe: add contingency (SVCAN) #3065

Closed

antoniospneto marked this pull request as draft May 15, 2024 19:40

antoniospneto force-pushed the nfe_transmission_with_contingency branch 7 times, most recently from 3e3f9b0 to b75f1ae Compare May 17, 2024 17:11

antoniospneto changed the title ~~[IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2)~~ [14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) May 22, 2024

antoniospneto force-pushed the nfe_transmission_with_contingency branch 4 times, most recently from 216920f to f7381ed Compare May 28, 2024 01:24

antoniospneto added 2 commits May 28, 2024 23:06

[REF] l10n_br_fiscal: fix typos; improve wording

0fd1a42

[IMP] l10n_br_fiscal: adds support for sync communcation

2d1502a

antoniospneto force-pushed the nfe_transmission_with_contingency branch from f7381ed to b2d665a Compare May 29, 2024 02:06

antoniospneto marked this pull request as ready for review May 29, 2024 02:12

rvalyi reviewed May 30, 2024

View reviewed changes

l10n_br_nfe/tests/test_nfe_webservices.py Show resolved Hide resolved

marcelsavegnago requested review from mileo May 31, 2024 11:49

felipemotter approved these changes May 31, 2024

View reviewed changes

antoniospneto added 2 commits May 31, 2024 09:22

[REF] l10n_br_account_nfe: rename nfce contingency function

3aef77f

antoniospneto force-pushed the nfe_transmission_with_contingency branch from b2d665a to 3aef77f Compare May 31, 2024 12:22

antoniospneto added a commit to Engenere/l10n-brazil that referenced this pull request Jun 1, 2024

Merge remote-tracking branch 'eng/nfe_transmission_with_contingency' …

70e3d75

…into 14.0-merged-features2 OCA#3081

rvalyi approved these changes Jun 11, 2024

View reviewed changes

antoniospneto mentioned this pull request Jun 18, 2024

[14.0][IMP] l10n_br_fiscal: force workflow transtition #3130

Open

rvalyi mentioned this pull request Jun 20, 2024

[FIX][14.0] l10n_br_nfe, l10n_br_fiscal: multiple document types #2728

Open

rvalyi requested changes Jun 20, 2024

View reviewed changes

l10n_br_nfe/models/document.py Show resolved Hide resolved

OCA-git-bot removed approved ready to merge labels Jun 20, 2024

rvalyi approved these changes Jun 28, 2024

View reviewed changes

OCA-git-bot added approved ready to merge labels Jun 28, 2024

rvalyi mentioned this pull request Jul 5, 2024

[14.0][IMP] l10n_br_fiscal: Resolvendo o problema de cálculo da estimativa de imposto nacional e importada #3152

Closed

mileo approved these changes Jul 10, 2024

View reviewed changes

OCA-git-bot added the bot is merging ⏳ label Jul 10, 2024

OCA-git-bot merged commit d7ba770 into OCA:14.0 Jul 10, 2024
7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Jul 10, 2024

mileo mentioned this pull request Jul 11, 2024

[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: atualização do parâmetro do tipo de emissão na geração da chave edoc #2395

Draft

This was referenced Jul 21, 2024

[15.0][MIG][l10n_br_nfe] #2874

Merged

[16.0][l10n_br_fiscal] Oca port from 14.0 to 16.0 pr 3081 #3200

Merged

[15.0][l10n_br_fiscal] Oca port from 14.0 to 15.0 pr 3081 #3192

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081

[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081

antoniospneto commented May 15, 2024 •

edited

Loading

OCA-git-bot commented May 15, 2024

rvalyi commented May 15, 2024 •

edited

Loading

rvalyi left a comment •

edited

Loading

renatonlima commented Jun 12, 2024

rvalyi commented Jun 12, 2024

mileo commented Jun 12, 2024

rvalyi commented Jun 12, 2024

rvalyi commented Jun 16, 2024

mileo commented Jun 20, 2024

OCA-git-bot commented Jun 28, 2024

rvalyi commented Jul 4, 2024

mileo commented Jul 5, 2024

mileo commented Jul 10, 2024

OCA-git-bot commented Jul 10, 2024

OCA-git-bot commented Jul 10, 2024

[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081

[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081

Conversation

antoniospneto commented May 15, 2024 • edited Loading

OCA-git-bot commented May 15, 2024

rvalyi commented May 15, 2024 • edited Loading

rvalyi left a comment • edited Loading

Choose a reason for hiding this comment

renatonlima commented Jun 12, 2024

rvalyi commented Jun 12, 2024

mileo commented Jun 12, 2024

rvalyi commented Jun 12, 2024

rvalyi commented Jun 16, 2024

mileo commented Jun 20, 2024

OCA-git-bot commented Jun 28, 2024

rvalyi commented Jul 4, 2024

mileo commented Jul 5, 2024

mileo commented Jul 10, 2024

OCA-git-bot commented Jul 10, 2024

OCA-git-bot commented Jul 10, 2024

antoniospneto commented May 15, 2024 •

edited

Loading

rvalyi commented May 15, 2024 •

edited

Loading

rvalyi left a comment •

edited

Loading