-
Notifications
You must be signed in to change notification settings - Fork 340
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
Bank of America #209
base: master
Are you sure you want to change the base?
Bank of America #209
Conversation
Bank of America |
OPa @fdalsotto Você já conseguiu validar essa implementação direto com o banco? |
sim @angeliski - isso ja esta' em prod ha' mais de um ano |
Show de bola @fdalsotto ! |
formatacao de numeros impressos no boleto deixa a desejar para casos aonde o valor e' maior do que R$999,99 porque falta o separador de casas decimais. Me avisa se precisar de uma classe para testar o boleto e eu te passo a classe que eu estava testando |
oi @fdalsotto, @angeliski tudo bem? parece uma boa sim, só faltaram os testes de unidade assim como o dos outros bancos: assim que estiverem ok podemos mergear (; |
eu nao estava aware dos testes unitarios e nesse momento nao faz sentido eu
faze-los porque como falei, isso ja esta em prod ha' um bom tempo.
@angeliski <https://github.com/angeliski> vc tem como criar as classes para
fecharmos esse revision? Grato
2017-11-07 15:02 GMT+00:00 Rodrigo Turini <[email protected]>:
… oi @fdalsotto <https://github.com/fdalsotto>, @angeliski
<https://github.com/angeliski>
tudo bem?
parece uma boa sim, só faltaram os testes de unidade assim como o dos
outros bancos:
https://github.com/caelum/caelum-stella/blob/master/
stella-boleto/src/test/java/br/com/caelum/stella/boleto/
bancos/BancoDoBrasilTest.java
assim que estiverem ok podemos mergear (;
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#209 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AQ9NLXeiyPfnVLXrs5t-8lY5m9_NY2Taks5s0HD4gaJpZM4MSwHR>
.
--
-------------------------------------
Kind Regards,
Fernando Ricardo Dal'Sotto
|
@fdalsotto Eu até posso criar os testes de unidade, mas vou ter que mandar pra vc por email, porque eu não tenho permissão de commit na sua branch. Pode ser? |
claro
2017-11-07 15:38 GMT+00:00 Rogerio Angeliski <[email protected]>:
… @fdalsotto <https://github.com/fdalsotto> Eu até posso criar os testes de
unidade, mas vou ter que mandar pra vc por email, porque eu não tenho
permissão de commit na sua branch. Pode ser?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#209 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AQ9NLZ0gnkhcNJUn2xzzqnNJXBqxap_Jks5s0HmSgaJpZM4MSwHR>
.
--
-------------------------------------
Kind Regards,
Fernando Ricardo Dal'Sotto
|
Essa Issue foi marcada automáticamente como obsoleta, devido a um longo periodo de inatividade. Se nenhuma interação ocorrer nos próximos dias, ela será encerrada. Agradecemos a sua contribuição, esse processo é apenas para manter o repositório mais organizado. |
Vou fazer esses testes no feriado. |
Disponibilizando Bank of America