[14.0][WIP] l10n_br_pos: hard refactor no módulo #2890
Draft
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
|
Hi @luismalta, @lfdivino, @mileo, |
ygcarvalh
force-pushed
the
ref/l10n_br_pos
branch
from
d300147
to
4f3b194
Compare
ygcarvalh
force-pushed
the
ref/l10n_br_pos
branch
from
4f3b194
to
111b335
Compare
ygcarvalh
force-pushed
the
ref/l10n_br_pos
branch
from
1d680ff
to
cc1b97d
Compare
|
pessoal, apenas para dizer que eu apoio a ideia desse refactor ta... |
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.
Esse é um trabalho que estou fazendo já tem um tempo e que consiste num hard refactor no módulo base. Este processo envolve a remoção de funcionalidades ou arquivos não utilizados, tornando o módulo como uma base para o PDV da localização mesmo. Boa parte da lógica atualmente está relacionada com a CF-e, o que deixava ele bastante dependente de ter um SAT para funcionar e a instalação dos módulos derivados (
l10n_br_pos_cfeoul10n_br_pos_nfce).Por enquanto os testes devem estar quebrados por conta das coisas que eu fiz, mas estarei arrumando eles posteriormente e adicionando novos testes. Além disso, farei ajustes no arquivo
READMEpara fornecer instruções claras sobre como utilizar o módulo e realizar configurações.O objetivo central desta refatoração é aprimorar a lógica do código, facilitando potenciais migrações para versões mais recentes no futuro.
É provável que váriosPRs em andamento, com alterações nestes módulos, sejam encerrados e fiquem focados nos seus respectivos PR's de refatoração.
Estou aberto a sugestões e feedbacks durante este processo. Assim que o trabalho estiver pronto para revisão, abrirei o PR para revisão e realizarei um squash dos commits.
PR's que poderão ser fechados no futuro:
[WIP]