[14.0][IMP] l10n_br_nfe: Emissão da NFCe #2329
Conversation
|
Hi @rvalyi, @renatonlima, |
mileo
changed the title
ygcarvalh
force-pushed
the
feature/nfce
branch
from
cefa7f0
to
95a8015
Compare
ygcarvalh
changed the title
ygcarvalh
force-pushed
the
feature/nfce
branch
3 times, most recently
from
4629885
to
26d7a74
Compare
ygcarvalh
force-pushed
the
feature/nfce
branch
2 times, most recently
from
ee64f7a
to
4c7ab4e
Compare
|
Adicionado dependência das alterações do PR #2395. |
|
LGTM |
ygcarvalh
changed the title
|
@lfdivino @felipezago.. está com erro nos testes e no runboat.. Está aprovado mesmo ? |
|
parece que tem um erro no runboat mas acredito que a origem pode ser de outro lugar.. vi o mesmo erro em outra PR se nao me engano |
|
@marcelsavegnago o erro nos testes eu estou investigando o que pode ter sido, o do runbot tinha dado em outro PR meu e localmente tentando subir com dado de demo... |
Eh então eu vi que aconteceu em outras PRs .. vlwww |
ygcarvalh
force-pushed
the
feature/nfce
branch
2 times, most recently
from
8d1132e
to
66e4de3
Compare
felipezago
force-pushed
the
feature/nfce
branch
from
19d730b
to
955d40e
Compare
l10n_br_nfe/models/document.py
Outdated
|
|
||
| if infProt.cStat in AUTORIZADO: | ||
| try: | ||
| record.make_pdf() |
There was a problem hiding this comment.
Não seria melhor criar o PDF em outro momento ou em segundo plano ?
Mesmo fazendo o tratamento da exceção para não interromper o fluxo, essa é uma operação muito critica.
Se por algum motivo a requisição demorar muito, o odoo vai "matar" essa requisição por esperar muito tempo.
Então penso que quanto menos processamento a gente fizer nessa hora melhor, querendo ou não essa geração do PDF é bem pesada.
There was a problem hiding this comment.
Também não vejo motivo para gerarmos o PDF durante a transmissão.
felipezago
force-pushed
the
feature/nfce
branch
from
47ec5a6
to
b681f0d
Compare
felipezago
force-pushed
the
feature/nfce
branch
from
b681f0d
to
91e5332
Compare
|
This PR has the |
|
/ocabot merge major |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 9c0eb61. Thanks a lot for contributing to OCA. ❤️ |
Bom, fiz algumas alterações no módulo
l10n_br_nfevisando a emissão da NFCe, mas sem a necessidade de alterar os fluxos das coisas que já funcionam atualmente. A remoção de algumas tags (como IPI e a data e hora de entrada/saída) vão ser alteradas apenas quando o tipo de documento selecionado for o 65. Outro ponto foi em relação ao fluxo, como a NFCe pode ter ambos os casos (síncrono e assíncrono), foi necessário realizar uma validação no retorno para lidar com esse caso.Houve também uma alteração realizada na biblioteca do
erpbrasil.edocque adiciona uma nova classe da NFCe, assim como uma validação para criarmos esse objeto quando o tipo de documento selecionado for esse.Além disso, uma nova configuração foi adicionada a empresa. A NFCe precisa, obrigatoriamente, do CSC Token e do Código emitido pelo próprio SEFAZ, além da versão do QR-Code (1 ou 2).
Próximos passos:
Depende das alterações que foram realizadas no PR erpbrasil/erpbrasil.edoc#51
Sugestões, dúvidas e críticas são sempre bem-vindas.