[12.0] Fiscal Events and Wizards Refactoring #983
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
renatonlima
force-pushed
the
12.0-fiscal-events-wizards
branch
from
367ac46
to
1fd0db2
Compare
This was referenced Oct 26, 2020
|
@renatonlima pode fazer um rebase deste PR para revisarmos plz. Vou dar prioridade no merge do #1007 por hora. |
|
UP |
|
realmente isso virou o proximo assunto do Renato depois que ele deixou as compras no ponto. |
|
UP :) |
renatonlima
force-pushed
the
12.0-fiscal-events-wizards
branch
from
ea8734a
to
7135eb0
Compare
renatonlima
force-pushed
the
12.0
branch
5 times, most recently
from
7b2f6a6
to
6100848
Compare
|
@renatonlima consegue fazer um rebase desta PR fazendo favor ? |
|
Pessoal eu vou fazer o rebase deste PR mas antes eu vou finalizar o PR #1099 |
This was referenced Mar 10, 2021
renatonlima
force-pushed
the
12.0-fiscal-events-wizards
branch
from
3a7f79a
to
57acd47
Compare
|
@renatonlima pode dar uma olhada nos testes plz? |
|
@renatonlima pode dar uma olhada nos erros do travis ? |
|
Bom, agora que foi feito o merge da NFe parece que este aqui seria o mais importante a ser mesclado também. Grande @renatonlima, acredita que consegue mexer em algo nesta semana ? (perguntando somente para que possamos nos programar) |
|
Replaced by #1300 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Olá pessoal,
Como hoje estamos limpando os módulos dos documentos fiscais (l10n_br_spec_nfe, l10n_br_nfe, l10n_br_nfse) para resolver alguns problemas de usabilidade para dá suporte a todos os tipos de documentos fiscais, este PR refatora alguns objetos:
Eventos Fiscais, hoje os eventos fiscais tem uma classe abstrata que é usada para implementar os eventos de autorização, cancelamento, carta de correção, inutilização e etc... para cada tipo de evento esta sendo criado um objeto concreto, foi refatorado para ter um mixin da classe de eventos que é implementado em uma unica classe concreta de eventos, desta forma podemos ver todos os eventos de um documento fiscal, assim como ao consultar na sefaz;
Refatorar os wizards genericos dos eventos de cancelemento, carta de correção, inutilização de numeração;
Criação do wizard generico e reutilizável de importação do documento fiscal (para facilitar a importação nos modulos de l10n_br_nfe, l10n_br_nfse e etc);
Refatorar os eventos de manifestação de destinatário (MD-e) que deveria ser um evento fiscal do documento e não outro objeto;
eliminar o campo state_edoc e state, remover campos duplicados nos objetos document.workflow e document.eletronic e refatorar métodos de eventos dos documentos fiscais para diminuir a quantidade de codigos repetidos;
Adicionado na serie de documento fiscal um campo One2many de numerações invalidadas:
Removido o campo document_section;
Removido campo duplicado is_edoc_printed;