Altera images para PNG por padrão

[rodrigondec]

Resumo

Flexibiliza os métodos auxiliares dos services para possibilitar uma utilização mais modular.

Troca a geração de imagens base64 de SVG para PNG por padrão. Essa alteração afeta o código de barras dos boletos e QR Code do PIX.

[peidrao]

Uma questão antes de revisar e entender o contexto. PNG se torna melhor que esses outros formatos?

[rodrigondec]

Uma questão antes de revisar e entender o contexto. PNG se torna melhor que esses outros formatos?

Emails não aceitam renderização de imagens SVG como attachment inline.

Para que seja feito a renderização do código de barras/qrcode PIX no email de cobrança do boleto é necessário fazer a imagem PNG, e não SVG.

E pelo que eu vi, o PNG ocupa alguns vários caracteres a menos. Alguns KB a menos na requisição então!

[peidrao]

Uma questão antes de revisar e entender o contexto. PNG se torna melhor que esses outros formatos?

Emails não aceitam renderização de imagens SVG como attachment inline.

Para que seja feito a renderização do código de barras/qrcode PIX no email de cobrança do boleto é necessário fazer a imagem PNG, e não SVG.

E pelo que eu vi, o PNG ocupa alguns vários caracteres a menos. Alguns KB a menos na requisição então!

O email entra como span com svg?

[rodrigondec]

Uma questão antes de revisar e entender o contexto. PNG se torna melhor que esses outros formatos?

Emails não aceitam renderização de imagens SVG como attachment inline.
Para que seja feito a renderização do código de barras/qrcode PIX no email de cobrança do boleto é necessário fazer a imagem PNG, e não SVG.
E pelo que eu vi, o PNG ocupa alguns vários caracteres a menos. Alguns KB a menos na requisição então!

O email entra como span com svg?

Só não renderiza mesmo.

[peidrao]

Para mim está aprovado, só tenho uma questão quanto aos testes, alguns estão sem nenhuma descrição. Creio que seria interessante destacar o que se espera ou o que ele está testando.

[PedroRegisPOAR]

LGTM

[codecov]

Codecov Report

Merging #60 (1ace180) into main (ed01fe1) will increase coverage by 0.23%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   89.18%   89.42%   +0.23%     
==========================================
  Files          23       23              
  Lines        1036     1059      +23     
==========================================
+ Hits          924      947      +23     
  Misses        112      112              

Files Changed	Coverage Δ
bb_wrapper/services/b64.py	100.00% <100.00%> (ø)
bb_wrapper/services/barcode.py	100.00% <100.00%> (ø)
bb_wrapper/services/qrcode.py	100.00% <100.00%> (ø)

Files Changed	Coverage Δ
bb_wrapper/services/b64.py	100.00% <100.00%> (ø)
bb_wrapper/services/barcode.py	100.00% <100.00%> (ø)
bb_wrapper/services/qrcode.py	100.00% <100.00%> (ø)