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

Corrige criação do QrCode PIX #27

Merged
merged 1 commit into from
Aug 20, 2021
Merged

Corrige criação do QrCode PIX #27

merged 1 commit into from
Aug 20, 2021

Conversation

rodrigondec
Copy link
Member

O nome do recebedor só comporta 13 caracteres.

Além disso, só aceita UNICODE!

@rodrigondec rodrigondec added Tamanho: P Tarefa pequena Tipo: Hotfix 🧑‍🚒 Está quebrado em prod e precisa ajeitar asap! labels Aug 20, 2021
@codecov
Copy link

codecov bot commented Aug 20, 2021

Codecov Report

Merging #27 (f313f42) into main (c385d9b) will increase coverage by 0.14%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   80.25%   80.39%   +0.14%     
==========================================
  Files          17       17              
  Lines         547      551       +4     
==========================================
+ Hits          439      443       +4     
  Misses        108      108              
Impacted Files Coverage Δ
bb_wrapper/wrapper/pix_cob.py 45.83% <ø> (ø)
bb_wrapper/services/pixcode.py 100.00% <100.00%> (ø)
Impacted Files Coverage Δ
bb_wrapper/wrapper/pix_cob.py 45.83% <ø> (ø)
bb_wrapper/services/pixcode.py 100.00% <100.00%> (ø)

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.

Aprovado!

@@ -61,10 +62,12 @@ def create(self, location: str, nome_recebedor: str):
data += BRCodeService().create_field_string(_id="58", value="BR")

# campo ID 59 Merchant Name
nome_recebedor = nome_recebedor[:13]
nome_recebedor = UnicodeService().parse_unicode_to_alphanumeric(nome_recebedor)
Copy link
Contributor

Choose a reason for hiding this comment

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

Vou dar uma olhada nesse UnicodeService, não lembrava que isso existia.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Tamanho: P Tarefa pequena Tipo: Hotfix 🧑‍🚒 Está quebrado em prod e precisa ajeitar asap!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants