[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: Melhorias na transmissão da NF-e (v2) #3081
Conversation
|
Hi @luismalta, @mileo, @gabrielcardoso21, @renatonlima, @ygcarvalh, @rvalyi, @AndreMarcos, @marcelsavegnago, |
|
fiz o merge de akretion/nfelib#103 Não sei se isso permite aos testes de rodar, vou mandar rodar... |
antoniospneto
force-pushed
the
nfe_transmission_with_contingency
branch
7 times, most recently
from
3e3f9b0
to
b75f1ae
Compare
antoniospneto
changed the title
antoniospneto
force-pushed
the
nfe_transmission_with_contingency
branch
4 times, most recently
from
216920f
to
f7381ed
Compare
antoniospneto
force-pushed
the
nfe_transmission_with_contingency
branch
from
f7381ed
to
b2d665a
Compare
- 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
antoniospneto
force-pushed
the
nfe_transmission_with_contingency
branch
from
b2d665a
to
3aef77f
Compare
…into 14.0-merged-features2 OCA#3081
There was a problem hiding this comment.
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?
|
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. |
|
@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. |
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. |
|
@mileo conseguiu ver se tinha algum problema para vcs neste PR? |
Estamos validando com um cliente esse pr hj. |
|
This PR has the |
|
@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. |
|
/ocabot merge major |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 3af68b3. Thanks a lot for contributing to OCA. ❤️ |
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:
Depende de:
Falta fazer: