Skip to content
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

Um fix para o BoletoItau, teste para ele no pyboleto_sample e algum rearranjo de comments #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dalembertian
Copy link

Olá,

Arrumei um pequeno ponto em BoletoItau (campo_livre deve ser str, e não unicode, de acordo com o que espera a super classe), e incluí dados para teste no pyboleto_sample, que agora também gera boletos para o Itaú. Nada muito significativo...

Abraço,
Rubens

@travisbot
Copy link

This pull request passes (merged 244a33e into 0802690).

@wiliamsouza
Copy link

O que falta para este pull request ser aceito?

@eduardocereto
Copy link
Owner

Eu tenho 2 problemas com esse pull request. O primeiro é um problema de documentação principalmente no arquivo data.py, o @dalembertian mudou o formato de algumas pydoc strings mas eles estavam dentro do padrão, então eu quero arrumar isso antes de comitar, mas isso eh o de menos.

Esse pull request também faz alterações no arquivo pdf.py. Este arquivo é comum a todas as implementações e eu quero me certificar que estas modifcações não causem problemas nos outros bancos.

Tirando isso eu estou ok com o patch. Os testes ainda passam então proavelmente não teremos nenhum problema aqui. Estou apenas esperando liberar um tempo para olhar para ele com mais cuidado e fazer o merge.

@wiliamsouza
Copy link

Opa pessoal,
testei o boelto itau carteira 157 pelo menos paguei um pelo santander e ele aceitou assim que tiver confirmação de recebimento aviso aqui.

@mercutio22
Copy link

Obrigado williamsouza!

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