8000 Create pdf for Santander registered boleto by rafaelgotts · Pull Request #1 · luizalabs/pyboleto · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Create pdf for Santander registered boleto #1

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 acc 8000 ount related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 6, 2017

Conversation

rafaelgotts
Copy link

No description provided.

self.instrucoes = kwargs.pop('instrucoes', [])
super(BoletoRegistradoSantander, self).__init__(**kwargs)

def nosso_numero_by_santander(self):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Que tal ser uma property? 🤔

def nosso_numero_by_santander(self):
""" On new Webservice for boleto,
santander can create their own
our_number with 13 and without dv

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não seria "nosso_numero" mesmo? Acho que não foi usado our_number em nenhum outro lugar.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eu estava usando our_numberem outro projeto e acabei me confundindo na hora de escrever essa parte. Vou arrumar.

@rafaelgottsfritz-luizalabs rafaelgottsfritz-luizalabs force-pushed the rg-include_registered_boleto branch from e7029f2 to fc46338 Compare June 5, 2017 22:51
@rafaelgottsfritz-luizalabs rafaelgottsfritz-luizalabs merged commit 3f1b0c3 into master Jun 6, 2017
@rafaelgottsfritz-luizalabs rafaelgottsfritz-luizalabs deleted the rg-include_registered_boleto branch June 6, 2017 13:47
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
0