Bank of America

[fdalsotto]

Disponibilizando Bank of America

[fdalsotto]

Bank of America

[angeliski]

OPa @fdalsotto Você já conseguiu validar essa implementação direto com o banco?

[fdalsotto]

sim @angeliski - isso ja esta' em prod ha' mais de um ano

[angeliski]

Show de bola @fdalsotto ! Estou tentando dar uma movimentada aqui, porque é um projeto muito bom e útil. Acredito que o @Turini ou o @mariofts possa nos ajudar nesse merge. :)

[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

[Turini]

oi @fdalsotto, @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 (;

[fdalsotto]

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 notifications@github.com:

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 https://github.com/caelum/caelum-stella/pull/209#issuecomment-342509504, or mute the thread https://github.com/notifications/unsubscribe-auth/AQ9NLXeiyPfnVLXrs5t-8lY5m9_NY2Taks5s0HD4gaJpZM4MSwHR .

--

Kind Regards, Fernando Ricardo Dal'Sotto

[angeliski]

@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?

[fdalsotto]

claro

2017-11-07 15:38 GMT+00:00 Rogerio Angeliski notifications@github.com:

@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 https://github.com/caelum/caelum-stella/pull/209#issuecomment-342521747, or mute the thread https://github.com/notifications/unsubscribe-auth/AQ9NLZ0gnkhcNJUn2xzzqnNJXBqxap_Jks5s0HmSgaJpZM4MSwHR .

--

Kind Regards, Fernando Ricardo Dal'Sotto

[stale[bot]]

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.

[angeliski]

Vou fazer esses testes no feriado.