Skip to content

Conversation

CristianoMafraJunior
Copy link
Member

Reabertura da PR #3724 pra v16.0

Objetivo da PR

Esta PR adiciona a funcionalidade de Alterar data de desconto do CNAB
image

@mbcosta estarei trazendo essa PR para v16 fiz as alterações que você pediu para trabalhar com o campo date, aqui deixei tudo em um commit mas se quiser posso separar a questão da lógica da informação do banco.

@OCA-git-bot
Copy link
Contributor

Hi @mbcosta,
some modules you are maintaining are being modified, check this out!

@CristianoMafraJunior CristianoMafraJunior force-pushed the 16.0-fix-data_desconto branch 2 times, most recently from 15ae072 to a521232 Compare August 16, 2025 02:11
@CristianoMafraJunior CristianoMafraJunior force-pushed the 16.0-fix-data_desconto branch 2 times, most recently from 8b269c0 to 29a4a70 Compare August 18, 2025 01:47
@CristianoMafraJunior CristianoMafraJunior marked this pull request as ready for review August 18, 2025 03:37
@mbcosta
Copy link
Contributor

mbcosta commented Sep 3, 2025

Valeu @CristianoMafraJunior . parabéns pelo trabalho, não parece ser necessário separar os commits, tem apenas os conflitos que surgiram que devem ser referentes a padronização das Visões e uma alteração no arquivo de demonstração das Configurações CNAB, acredito que é de simples solução mas se precisar de ajuda e só avisar, desculpe pela demora no retorno.

@CristianoMafraJunior
Copy link
Member Author

Valeu @CristianoMafraJunior . parabéns pelo trabalho, não parece ser necessário separar os commits, tem apenas os conflitos que surgiram que devem ser referentes a padronização das Visões e uma alteração no arquivo de demonstração das Configurações CNAB, acredito que é de simples solução mas se precisar de ajuda e só avisar, desculpe pela demora no retorno.

pronta pra revisão @mbcosta só uma observação ali eu passei # noqa: C901 no método do teste por que ele ta muito complexo não sei é algo que vai brekar mas ...

Copy link
Contributor

@mbcosta mbcosta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Valeu @CristianoMafraJunior , sobre o noqa: C901 é quantidade de IFs no método, como é referente ao teste não acho necessário bloquear o PR, eu já ia ver o teste desse caso de uso para manter a Cobertura dos Testes, como havia comentado no PR da v14, então eu já vejo de tentar resolver isso também em outro PR.

Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ficou bem legal, obrigado @CristianoMafraJunior

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants