-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update BancoSicredi.CNAB240.cs #335
Conversation
BoletoNet#334 - Ajustado identificação do titulo da empresa na remessa do sicred
@brunogoulart28 Sua alteracao não esta correta, o campo utilizado é o NumeroDocumento, este é que você precisa preencher na classe boleto. |
Será que isso não vai quebrar aplicações em andamento? Talvez, se realmente for necessário, o ideal seria criar uma configuração para manter o funcionando atual por padrão. Ou se o comportamento atual realmente estiver errado, seria melhor documentar o embasamento para não restar duvidas |
@brunogoulart28 Continua incorreto. veja o comentário acima. |
@carloscds entendo que você esteja confundindo as coisas. O campo citado, posição 196 a 220, não é o campo para o número do documento, mas sim para um código interno. A descrição para a posição indica: |
O NumeroDocumento vai na posição 63 tanto no Sicredi como no Banco Brasil: |
Identificacao do Documento na Empresa é o seu numero do sistema, como você controla o seu financeiro. Ele vem depois no retorno para que você possa, eventualmente, usar para baixa. |
Exatamente isso, mas no caso do SICREDI está retornando o NumeroDocumento pois é o que está sendo colocado na remessa na posição 196 ao invés de colocar o NumeroControleParticipante. Essa foi a correção implementada no PR do Bruno. |
Ok, Mas a mudança é na remessa e nao no retorno. A Remessa é o numero do documento, nao ? |
Nós precisamos enviar o NumeroControleParticipante na posição 196 do arquivo de REMESSA, para que o banco retorne esse mesmo número no arquivo de RETORNO. |
A maioria dos bancos já está dessa forma, não sei se todos, mas a implementação para o SICREDI está incorreta e diferente dos outros bancos. |
#334 - Ajustado identificação do titulo da empresa na remessa do sicred