chore: possibilita o envio de medias do tipo [svg, tiff] vindas do Chatwoot by KokeroO · Pull Request #1483 · EvolutionAPI/evolution-api · GitHub
Skip to content

chore: possibilita o envio de medias do tipo [svg, tiff] vindas do Chatwoot#1483

Merged
DavidsonGomes merged 3 commits intoEvolutionAPI:developfrom
KokeroO:develop
May 22, 2025
Merged

chore: possibilita o envio de medias do tipo [svg, tiff] vindas do Chatwoot#1483
DavidsonGomes merged 3 commits intoEvolutionAPI:developfrom
KokeroO:develop

Conversation

@KokeroO
Copy link
Copy Markdown
Contributor

@KokeroO KokeroO commented May 22, 2025

Descrição

Este PR adiciona a possiblidade de envio de arquivos do tipo SVG e TIFF da mesma forma que faz com do tipo GIF vindas do Chatwoot.
Atualmente o WhatsApp WEB somente aceita imagens do tipo JPG, PNG e WEBP de forma nativa, GIF e SVG o core processa as imagens em JPEG e PNG respectivamente.

Como o Chatwoot não entende que isso é um problema no canal de API deles, visto que o envio do arquivo é feito puro. Então sim, se entende ser um problema da API. Devemos tratar esses attachs na API Evolution.

Proposta

Enviar os tipos SVG e TIFF da mesma forma que atualmente é feito com as imagens do tipo GIF.

Solução

Adicionei uma constante com um array de extensões de mimetype: image e que deverão ser enviadas como tipo document. O array permite a fácil inclusão de outros tipos de imagens que poderão ser identificadas futuramente.

const documentExtensions = ['.gif', '.svg', '.tiff'];
if (type === 'image' && parsedMedia && documentExtensions.includes(parsedMedia?.ext)) {
  type = 'document';
}

Futuro

Futuramente poderíamos implementar um processador de imagens. Convertendo nos formatos aceitáveis pelo WhatsApp.

@sourcery-ai
Copy link
Copy Markdown
Contributor

sourcery-ai Bot commented May 22, 2025

@KokeroO
Copy link
Copy Markdown
Contributor Author

KokeroO commented May 22, 2025

@sourcery-ai review

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey @KokeroO - I've reviewed your changes - here's some feedback:

  • Consider hoisting the documentExtensions array out of the function (or module) scope to avoid recreating it on every call and centralize configuration.
  • You might also want to include the common .tif extension alongside .tiff to cover both naming variants.
Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@KokeroO
Copy link
Copy Markdown
Contributor Author

KokeroO commented May 22, 2025

@DavidsonGomes DavidsonGomes merged commit 6f47a54 into EvolutionAPI:develop May 22, 2025
1 check passed
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.

2 participants