Criação de transferência em lotes pelo PIX #54

peidrao · 2023-04-20T01:23:20Z

Resumo

Relacionado aos issues/prs:

Este PR adiciona a função de transferência em lote via pix

As novas ações permitem fazer a transferência por todos os meios (chave, email, documento, dados bancários)

Alterações de models

Sim
Não

Alterações de services

Sim
Não

Alterações de wrappers

Sim
Não

Foram adicionados dois novos wrappers um para criação de transferência via chave, documento cpf/cnpj (criar_transferencia_por_chave_pix) e outro para dados bancários (criar_transferencia_por_dados_bancarios_pix)

peidrao · 2023-04-20T01:24:12Z

Sera preciso criar exemplos para todos os casos? E-mail, chave aleatória, telefone e CPF/CNPJ

codecov · 2023-04-20T01:54:31Z

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (f048de5) 89.26% compared to head (a81a5b3) 91.45%.
Report is 1 commits behind head on main.

❗ Current head a81a5b3 differs from pull request most recent head 45f0277. Consider uploading reports for the commit 45f0277 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   89.26%   91.45%   +2.18%     
==========================================
  Files          23       24       +1     
  Lines        1062     1158      +96     
==========================================
+ Hits          948     1059     +111     
+ Misses        114       99      -15

Files	Coverage Δ
bb_wrapper/services/document.py	`100.00% <100.00%> (+16.66%)`	⬆️
bb_wrapper/services/pix.py	`97.22% <100.00%> (ø)`
bb_wrapper/wrapper/pagamento_lote.py	`96.15% <100.00%> (+0.87%)`	⬆️
bb_wrapper/wrapper/pix_cob.py	`52.54% <ø> (ø)`
bb_wrapper/models/pagamentos.py	`97.56% <93.75%> (+16.87%)`	⬆️

Files	Coverage Δ
bb_wrapper/services/document.py	`100.00% <100.00%> (+16.66%)`	⬆️
bb_wrapper/services/pix.py	`97.22% <100.00%> (ø)`
bb_wrapper/wrapper/pagamento_lote.py	`96.15% <100.00%> (+0.87%)`	⬆️
bb_wrapper/wrapper/pix_cob.py	`52.54% <ø> (ø)`
bb_wrapper/models/pagamentos.py	`97.56% <93.75%> (+16.87%)`	⬆️

brenomfviana

Faltou rodar make fmt, faz também testes com os outros tipos de chaves.

bb_wrapper/wrapper/pagamento_lote.py

examples/lotes_pagamento/data/transferencia_pix_chave_aleatoria.json

tests/wrapper/test_pagamentos.py

Co-authored-by: Breno Viana <[email protected]>

tests/wrapper/test_pagamentos.py

Co-authored-by: Breno Viana <[email protected]>

peidrao · 2023-04-20T17:33:35Z

Seria preciso criar algum modelo?

brenomfviana · 2023-04-20T19:27:14Z

Modelo de quê?

peidrao · 2023-04-20T19:55:38Z

Modelo de quê?

Na tarefa do clickup fala sobre criação de modelos e services, mas imagino que não seja necessário.

brenomfviana · 2023-04-20T19:58:59Z

Na tarefa do clickup fala sobre criação de modelos e services, mas imagino que não seja necessário.

Acho que não é necessário criar pra lib funcionar, mas é legal ter.

rodrigondec

Alguns detalhes sobre os argumentos e simplificações mais inteligentes

bb_wrapper/wrapper/pagamento_lote.py

bb_wrapper/models/pagamentos.py

examples/lotes_pagamento/transferencia_pix_dados_bancarios.py

…ta_pagamento

rodrigondec · 2024-01-30T21:01:12Z

bb_wrapper/models/pagamentos.py

+    cpf: Optional[str]
+    cnpj: Optional[str]


se não me engano tem que ser int mesmo!

Sim, a API BB é horrível nesse nível. Precisa transformar em int mesmo.

Ainda está como "str"!

bb_wrapper/models/pagamentos.py

bb_wrapper/wrapper/pagamento_lote.py

Sugestões feitas por rodrigo Co-authored-by: Rodrigo Castro <[email protected]>

bb_wrapper/models/pagamentos.py

rodrigondec · 2024-01-31T19:58:55Z

bb_wrapper/models/pagamentos.py

+    cpf: Optional[str]
+    cnpj: Optional[str]


Ainda está como "str"!

bb_wrapper/models/pagamentos.py

tests/wrapper/test_pagamentos.py

rodrigondec

O envio do PIX por dados bancários

Não deveria mandar o valor "cnpj" com dados vazio! Deveria enviar apenas um ou outro, e não os dois.

Rodei o exemplo com python -m examples.lotes_pagamento.transferencia_pix_dados_bancarios

DEBUG:urllib3.connectionpool:https://api.sandbox.bb.com.br:443 "POST /pagamentos-lote/v1/lotes-transferencias-pix?gw-dev-app-key=d27b377907ffabd01369e17df0050056b9f1a5b9 HTTP/1.1" 200 699
DEBUG:bb_wrapper.requests:response.request.body=b'{"numeroRequisicao": 9499946, "agenciaDebito": 1607, "contaCorrenteDebito": 99738672, "digitoVerificadorContaCorrente": "X", "tipoPagamento": 128, "listaTransferencias": [{"data": 1022024, "valor": 15.5, "tipoConta": 1, "agencia": 1234, "conta": 12345, "digitoVerificadorConta": "X", "formaIdentificacao": 5, "descricaoPagamento": "Uma transfer\\u00eancia via dados banc\\u00e1rios", "cpf": 28779295827, "cnpj": null, "numeroISPB": 360305}]}'


DEBUG:bb_wrapper.requests:{'numeroRequisicao': 9499946, 'estadoRequisicao': 1, 'quantidadeTransferencias': 1, 'valorTransferencias': 15.5, 'quantidadeTransferenciasValidas': 1, 'valorTransferenciasValidas': 15.5, 'listaTransferencias': [{'identificadorPagamento': 99499946731030000, 'data': 1022024, 'valor': 15.5, 'documentoDebito': 0, 'documentoCredito': 0, 'descricaoPagamento': 'Uma transferýncia via dados bancýrios', 'descricaoPagamentoInstantaneo': '', 'formaIdentificacao': 5, 'dddTelefone': 0, 'telefone': 0, 'email': '', 'cpf': 28779295827, 'cnpj': 0, 'identificacaoAleatoria': '', 'numeroCOMPE': 0, 'numeroISPB': 360305, 'tipoConta': 1, 'agencia': 1234, 'conta': 12345, 'digitoVerificadorConta': 'X', 'contaPagamento': '', 'indicadorMovimentoAceito': 'S', 'erros': []}]}

rodrigondec

PIX por chave

Estão sendo enviados todos os outros campos de chave como vazios também. "email", "cnpj", "identificacaoAleatoria", "telefone" + "dddTelefone"...

rodei com python -m examples.lotes_pagamento.transferencia_pix_cpf

DEBUG:bb_wrapper.requests:response.request.body=b'{"numeroRequisicao": 9499944, "agenciaDebito": 1607, "contaCorrenteDebito": 99738672, "digitoVerificadorContaCorrente": "X", "tipoPagamento": 128, "listaTransferencias": [{"data": 1022024, "valor": 7999.99, "formaIdentificacao": 3, "descricaoPagamento": "transferencia CPF", "dddTelefone": null, "telefone": null, "email": null, "cpf": 28779295827, "cnpj": null, "identificacaoAleatoria": null}]}'


DEBUG:bb_wrapper.requests:{'numeroRequisicao': 9499944, 'estadoRequisicao': 1, 'quantidadeTransferencias': 1, 'valorTransferencias': 7999.99, 'quantidadeTransferenciasValidas': 1, 'valorTransferenciasValidas': 7999.99, 'listaTransferencias': [{'identificadorPagamento': 99499944731030000, 'data': 1022024, 'valor': 7999.99, 'documentoDebito': 0, 'documentoCredito': 0, 'descricaoPagamento': 'transferencia CPF', 'descricaoPagamentoInstantaneo': '', 'formaIdentificacao': 3, 'dddTelefone': 0, 'telefone': 0, 'email': '', 'cpf': 28779295827, 'cnpj': 0, 'identificacaoAleatoria': '', 'numeroCOMPE': 0, 'numeroISPB': 0, 'tipoConta': 0, 'agencia': 0, 'conta': 0, 'digitoVerificadorConta': '', 'contaPagamento': '', 'indicadorMovimentoAceito': 'S', 'erros': []}]}

brenomfviana · 2024-07-04T17:18:00Z

bb_wrapper/models/pagamentos.py

        PixService().remove_null_values(values)
+        if "cnpj" not in values and "cpf" not in values:
+            raise ValueError("É necessário inserir o CNPj ou CPF")


Suggested change

raise ValueError("É necessário inserir o CNPj ou CPF")

raise ValueError("É necessário inserir o CNPJ ou CPF")

brenomfviana · 2024-07-04T17:18:20Z

bb_wrapper/wrapper/pagamento_lote.py

+        from bb_wrapper.services.document import DocumentoService, TipoInscricaoEnum
+        tipo_documento = DocumentoService().identifica_tipo(documento_favorecido)


brenomfviana · 2024-07-04T17:26:14Z

tests/services/test_pix.py

+        ):
+
+            expected = {"cnpj": 3794722000153}


Suggested change

):

expected = {"cnpj": 3794722000153}

):

expected = {"cnpj": 3794722000153}

peidrao added 3 commits April 19, 2023 22:02

criar transferencia pix

9fbd40b

testes de transferencias pix

2306997

exemplo de transferencia pix

e1c6b24

peidrao requested review from rodrigondec, brenomfviana, PedroRegisPOAR and arianesc April 20, 2023 01:23

formatacao e remocao de pdb

5121f2d

brenomfviana requested changes Apr 20, 2023

View reviewed changes

peidrao and others added 7 commits April 20, 2023 12:14

Update tests/wrapper/test_pagamentos.py

cbcc864

Co-authored-by: Breno Viana <[email protected]>

Update tests/wrapper/test_pagamentos.py

dca8256

Co-authored-by: Breno Viana <[email protected]>

Update tests/wrapper/test_pagamentos.py

33b6c34

Co-authored-by: Breno Viana <[email protected]>

fmt

216dd29

adicao de novos escopo de autorizacao do pix

cec80e5

fix testes

dbf4d76

fmt

558871d

brenomfviana reviewed Apr 20, 2023

View reviewed changes

tests/wrapper/test_pagamentos.py Outdated Show resolved Hide resolved

Update tests/wrapper/test_pagamentos.py

e9d83cc

Co-authored-by: Breno Viana <[email protected]>

adicao de exemplo de trasnferencia usando cpf

f088f9e

brenomfviana self-requested a review April 20, 2023 19:10

peidrao closed this Apr 20, 2023

peidrao reopened this Apr 20, 2023

adicao de wrapper para consulta de pix

7168239

rodrigondec requested changes Apr 20, 2023

View reviewed changes

bb_wrapper/wrapper/pagamento_lote.py Outdated Show resolved Hide resolved

bb_wrapper/wrapper/pagamento_lote.py Outdated Show resolved Hide resolved

bb_wrapper/wrapper/pagamento_lote.py Outdated Show resolved Hide resolved

peidrao added 3 commits January 30, 2024 15:08

adiciona serviço de transferencia pix via dados bancários no bb-wrapper

e58dd32

adiciona testes unitários

012dccc

adiciona example de chamada de requisição

31cc8ae

peidrao requested review from brenomfviana and rodrigondec January 30, 2024 18:10

rodrigondec requested changes Jan 30, 2024

View reviewed changes

peidrao added 3 commits January 30, 2024 17:09

adiciona numero ISPB

c2ff8f4

adiciona valores bancários como opcionais

f59ddc5

cria exemplo de transferencia pix via dados bancários apenas pela con…

8934d56

…ta_pagamento

rodrigondec requested changes Jan 30, 2024

View reviewed changes

Apply suggestions from code review

310a6e8

Sugestões feitas por rodrigo Co-authored-by: Rodrigo Castro <[email protected]>

peidrao commented Jan 30, 2024

View reviewed changes

bb_wrapper/models/pagamentos.py Outdated Show resolved Hide resolved

ajustes

a81a5b3

peidrao requested a review from rodrigondec January 31, 2024 01:41

rodrigondec requested changes Jan 31, 2024

View reviewed changes

peidrao added 3 commits January 31, 2024 17:43

ajustar modelos

446e6df

ajustar testes em wrapper

e462efa

ajustar testes

3ef005e

rodrigondec requested changes Feb 1, 2024

View reviewed changes

rodrigondec and others added 2 commits February 1, 2024 15:17

adiciona logging no exemplo!

ded5afb

ajustes

d1ea125

peidrao requested a review from rodrigondec February 1, 2024 20:33

peidrao and others added 2 commits February 2, 2024 10:39

removendo valores nulos

9f62125

adiciona tratamento do COMPE e cpf/cnpj dos dados.

45f0277

peidrao removed request for PedroRegisPOAR and arianesc February 7, 2024 12:29

ajustes

28f6829

brenomfviana reviewed Jul 4, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Criação de transferência em lotes pelo PIX #54

Criação de transferência em lotes pelo PIX #54

peidrao commented Apr 20, 2023 •

edited

Loading

peidrao commented Apr 20, 2023 •

edited

Loading

codecov bot commented Apr 20, 2023 •

edited

Loading

brenomfviana left a comment

peidrao commented Apr 20, 2023

brenomfviana commented Apr 20, 2023

peidrao commented Apr 20, 2023

brenomfviana commented Apr 20, 2023

rodrigondec left a comment

rodrigondec Jan 30, 2024

rodrigondec Jan 31, 2024

rodrigondec Jan 31, 2024

rodrigondec left a comment

rodrigondec left a comment

brenomfviana Jul 4, 2024

brenomfviana Jul 4, 2024

brenomfviana Jul 4, 2024

	raise ValueError("É necessário inserir o CNPj ou CPF")
	raise ValueError("É necessário inserir o CNPJ ou CPF")

		from bb_wrapper.services.document import DocumentoService, TipoInscricaoEnum
		tipo_documento = DocumentoService().identifica_tipo(documento_favorecido)

Criação de transferência em lotes pelo PIX #54

Are you sure you want to change the base?

Criação de transferência em lotes pelo PIX #54

Conversation

peidrao commented Apr 20, 2023 • edited Loading

Resumo

Alterações de models

Alterações de services

Alterações de wrappers

peidrao commented Apr 20, 2023 • edited Loading

codecov bot commented Apr 20, 2023 • edited Loading

Codecov Report

brenomfviana left a comment

Choose a reason for hiding this comment

peidrao commented Apr 20, 2023

brenomfviana commented Apr 20, 2023

peidrao commented Apr 20, 2023

brenomfviana commented Apr 20, 2023

rodrigondec left a comment

Choose a reason for hiding this comment

rodrigondec Jan 30, 2024

Choose a reason for hiding this comment

rodrigondec Jan 31, 2024

Choose a reason for hiding this comment

rodrigondec Jan 31, 2024

Choose a reason for hiding this comment

rodrigondec left a comment

Choose a reason for hiding this comment

O envio do PIX por dados bancários

rodrigondec left a comment

Choose a reason for hiding this comment

PIX por chave

brenomfviana Jul 4, 2024

Choose a reason for hiding this comment

brenomfviana Jul 4, 2024

Choose a reason for hiding this comment

brenomfviana Jul 4, 2024

Choose a reason for hiding this comment

peidrao commented Apr 20, 2023 •

edited

Loading

peidrao commented Apr 20, 2023 •

edited

Loading

codecov bot commented Apr 20, 2023 •

edited

Loading