l10n-brazil
l10n-brazil copied to clipboard
OCA
[14.0][IMP] l10n_br_account_payment_brcobranca: Aprimoramento nos laçamentos das tarifas
Objetivo
Esta PR propõe aprimorar o tratamento de tarifas cobradas pelos bancos, assegurando o registro de todas as tarifas identificadas no retorno bancário, independentemente da ocorrência à qual estão vinculadas. Antes desta atualização, o sistema registrava tarifas apenas nas ocorrências de liquidação. A mudança visa capturar e registrar de forma abrangente todas as tarifas associadas a diferentes tipos de ocorrências bancárias.
Exemplos de Casos Previamente Não Cobertos:
- "Confirmação de Entrada de Boleto" – Bancos que aplicam tarifas já no registro, não apenas na liquidação.
- No Ailos, a ocorrência "23 - Remessa à Cartório (Aponte em Cartório)" acarreta uma tarifa de R$ 17,00.
A ocorrência de tarifas depende do banco e do acordo firmado com o cliente. Existem múltiplas possibilidades de cobranças de tarifas, como alterações de vencimento, abatimentos e baixas. Não é necessário que o nome da ocorrência contenha explicitamente a palavra "TARIFA" para que uma tarifa seja aplicada.
Observações
-
Um novo campo, floating_days, foi adicionado ao diário do banco para indicar o número de dias necessários para o banco processar os recebimentos. Isso porque, no caso de uma tarifa sem liquidação de cobrança, muitas vezes não dispomos de uma data de liquidação ou de crédito/debito para usar como referência. Assim, na ausência da data de liquidação, a tarifa será registrada na data da ocorrência somada aos dias de flutuação.
-
Adicionamos diversos testes unitários para prevenir regressões.
-
Esta PR está em modo rascunho enquanto validamos as mudanças em ambiente de produção.
cc @CristianoMafraJunior @rvalyi @mbcosta @marcelsavegnago @mileo
Hi @rvalyi, @mbcosta, some modules you are maintaining are being modified, check this out!
@antoniospneto vi que o @CristianoMafraJunior aprovou. Mas ainda ta de rascunho ou ta OK para revisão do lado de vcs?
opa @rvalyi a principio tá tudo certo, só to testando um pouco em produção, vou esperar mais uns dois dias, se não tiver nenhum chamado vou por a pr aqui como pronta, valeu!
Pessoal, eu encontrei um pequeno problema, no itaú o valor da tarifa não tá somado ao valor pago pelo cliente, fazendo com que sempre fique um saldo em aberto no contas a receber, vamos fazer a correção ok
Olá, pessoal.
Gostaria de informar que a melhoria desta PR tem se mantido estável em produção nos últimos dias, sem registrar nenhum problema. Segue abaixo um retorno processado que inclui as tarifas:
Agora a PR está pronta para ser revisada. Se alguém tiver alguma dúvida ou quiser fazer sugestões, sinta-se à vontade para expressar suas opiniões. A feedback de vocês é sempre valioso.
Aqui um exemplo da uma tarifa cobrada na baixa simples:
Sem a PR o valor da tarifa não era exibido nesses casos, ficava sempre zerado.
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
valeu @antoniospneto , alguns pontos:
-
Tem alguma justificativa alterar os testes do caso UNICRED? Os arquivos de retorno do teste possuem valor na Tarifa no caso da Confirmação ou o que levou a essa alteração? Porque o ideal é manter os testes desse caso como estavam e apenas incluir os novos casos de Cobrança de Tarifa,
-
É importante que os testes desses casos de Tarifa sejam completos, isso é até a Reconciliação completa da Fatura, teria que adicionar os arquivos de Liquidação para confirmar se isso ocorre sem erros como você mesmo pontou
"Pessoal, eu encontrei um pequeno problema, no itaú o valor da tarifa não tá somado ao valor pago pelo cliente, fazendo com que sempre fique um saldo em aberto no contas a receber, vamos fazer a correção ok"
Isso evitaria esse problema e também evita futuras regressões. Seria bom que esse arquivo de Liquidação seja baseado em um real apenas para confirmar uma questão inicial que é esse Valor da Tarifa é informado e cobrado já nesse momento ou apenas informado e no momento da liquidação é efetivamente cobrado, para confirmar que o Valor da Tarifa não está sendo duplicado, acredito que é o caso de já ser cobrado inicialmente mas é importante a confirmação para tirar qualquer dúvida
Ainda esta em draft?
Sim, ainda não tá legal para entrar na branch principal, eu penso em simplificar a proposta, remover a parte do lançamento contábil da tarifa e deixar apenas o registro delas que antes não era exibido no retorno e nem associado ao contas a receber.
Também tenho que ver as questões que o @mbcosta comentou.
Em breve pretendo voltar nesse assunto.