Skip to content
This repository has been archived by the owner on Jan 24, 2025. It is now read-only.

Adiciona suporte a transferências para contas pagamento #66

Merged
merged 17 commits into from
Aug 2, 2024

Conversation

brenomfviana
Copy link
Contributor

No description provided.

@brenomfviana brenomfviana added Tipo: Funcionalidade 💎 Proposta ou funcionalidade Domínio: Back 🦇 Tarefa que envolve a equipe de backend labels Nov 30, 2023
@brenomfviana brenomfviana self-assigned this Nov 30, 2023
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

Attention: Patch coverage is 92.30769% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 89.33%. Comparing base (55408e4) to head (d974a94).
Report is 4 commits behind head on main.

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

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   89.45%   89.33%   -0.12%     
==========================================
  Files          23       23              
  Lines        1062     1069       +7     
==========================================
+ Hits          950      955       +5     
- Misses        112      114       +2     
Files Coverage Δ
bb_wrapper/models/pagamentos.py 80.89% <100.00%> (+0.21%) ⬆️
bb_wrapper/wrapper/request.py 79.81% <100.00%> (-1.67%) ⬇️
bb_wrapper/wrapper/pagamento_lote.py 95.49% <87.50%> (+0.21%) ⬆️
Files Coverage Δ
bb_wrapper/models/pagamentos.py 80.89% <100.00%> (+0.21%) ⬆️
bb_wrapper/wrapper/request.py 79.81% <100.00%> (-1.67%) ⬇️
bb_wrapper/wrapper/pagamento_lote.py 95.49% <87.50%> (+0.21%) ⬆️

peidrao
peidrao previously approved these changes Nov 30, 2023
Copy link
Contributor

@PedroRegisPOAR PedroRegisPOAR left a comment

Choose a reason for hiding this comment

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

LGTM

rodrigondec
rodrigondec previously approved these changes Nov 30, 2023
peidrao
peidrao previously approved these changes Nov 30, 2023
Copy link
Contributor

@PedroRegisPOAR PedroRegisPOAR left a comment

Choose a reason for hiding this comment

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

fmt tá quebrado

@brenomfviana brenomfviana dismissed stale reviews from peidrao and rodrigondec via 754c69c December 1, 2023 14:03
PedroRegisPOAR
PedroRegisPOAR previously approved these changes Dec 1, 2023
Copy link
Contributor

@PedroRegisPOAR PedroRegisPOAR left a comment

Choose a reason for hiding this comment

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

🚀

peidrao
peidrao previously approved these changes Dec 1, 2023
Copy link
Member

@rodrigondec rodrigondec left a comment

Choose a reason for hiding this comment

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

🚀

rodrigondec
rodrigondec previously approved these changes Mar 8, 2024
Co-authored-by: Rodrigo Castro <rodrigondec@gmail.com>
peidrao
peidrao previously approved these changes Mar 8, 2024
Copy link
Member

@rodrigondec rodrigondec left a comment

Choose a reason for hiding this comment

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

Revisão atualizada. Aguardando testes de integração no ambiente de produção

@brenomfviana brenomfviana removed NÃO MERGEAR 🛑 Mesmo que tudo esteja verde, não faça o merge deste PR Domínio: Back 🦇 Tarefa que envolve a equipe de backend labels Aug 2, 2024
@rodrigondec rodrigondec merged commit d3bcb35 into main Aug 2, 2024
2 checks passed
@rodrigondec rodrigondec deleted the feature/adiciona-suporte-conta-pagamento branch August 2, 2024 18:44
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Tipo: Funcionalidade 💎 Proposta ou funcionalidade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants