chore(banks): atualiza logo-urls.json para o logos-bancos-br 0.8.1 by rzmt · Pull Request #928 · BrasilAPI/BrasilAPI · GitHub
Skip to content

chore(banks): atualiza logo-urls.json para o logos-bancos-br 0.8.1 - #928

Open
rzmt wants to merge 1 commit into
BrasilAPI:mainfrom
rzmt:chore/logo-urls-0.8.1
Open

rzmt wants to merge 1 commit into
BrasilAPI:mainfrom
rzmt:chore/logo-urls-0.8.1

Conversation

@rzmt

@rzmt rzmt commented Oct 19, 2026

Copy link
Copy Markdown

📋 Descrição

O services/banco-central/logo-urls.json ainda é o mapa do logos-bancos-br 0.6.2 que entrou no #886. Este PR adiciona as 27 chaves que o mapa ganhou até a 0.8.1. Com isso, 21 instituições do /banks/v1 deixam de vir com logo_url: null, entre elas BNDES (007), Banco Original (212), HSBC (269), Daycoval (707), BNP Paribas (752) e Bank of America (755). As outras chaves são de instituições que só participam do Pix e não aparecem no endpoint.

Nenhuma URL existente muda e nenhum ISPB sai.

🎯 Tipo de Mudança

  • 🐛 Correção de bug (mudança que corrige um problema)
  • ✨ Nova funcionalidade (mudança que adiciona funcionalidade)
  • 💥 Breaking change (correção ou funcionalidade que causa quebra de compatibilidade)
  • 📝 Documentação (mudanças apenas em documentação)
  • ♻️ Refatoração (mudança que não corrige bug nem adiciona funcionalidade)
  • ⚡ Performance (mudança que melhora performance)
  • ✅ Testes (adiciona ou corrige testes)
  • 🔧 Configuração (mudanças em configuração ou build)

⚠️ Checklist de Compatibilidade (CRÍTICO)

  • ✅ Não remove campos de respostas de API existentes
  • ✅ Não renomeia campos de respostas de API existentes
  • ✅ Não muda tipos de dados de campos existentes (string → number, etc.)
  • ✅ Não muda o formato de URLs de endpoints existentes
  • ✅ Não muda códigos de status HTTP de endpoints existentes
  • ✅ Se fez mudanças incompatíveis, criei uma nova versão (v2, v3, etc.)

📚 Checklist de Documentação

  • ✅ Atualizei ou criei documentação OpenAPI em /pages/docs/doc/
  • ✅ Documentação inclui exemplos de requisição e resposta
  • ✅ Documentação está em português
  • ✅ Atualizei README.md se necessário
  • ✅ N/A - Mudanças não requerem documentação

🧪 Checklist de Testes

  • ✅ Criei ou atualizei testes E2E
  • ✅ Todos os testes passam localmente (npm test)
  • ✅ Teste de CORS funciona corretamente
  • ✅ Testei casos de erro (404, 400, 500, etc.)
  • ✅ Testei casos de sucesso
  • ✅ N/A - Mudanças não requerem testes

💻 Checklist de Código

  • ✅ Código segue os padrões do projeto (ESLint + Prettier)
  • ✅ Executei npm run fix antes de commitar
  • ✅ Não adicionei dependências desnecessárias ou pesadas
  • ✅ Código não expõe credenciais ou informações sensíveis
  • ✅ Validei todos os inputs de usuário
  • ✅ Tratei erros apropriadamente
  • ✅ Usei Conventional Commits

🚀 Checklist de Performance e Custos

  • ✅ Não adicionei processamento pesado que aumenta custos
  • ✅ Usei cache quando apropriado
  • ✅ Minimizei chamadas a APIs externas
  • ✅ Considerei impacto em rate limits de APIs externas
  • ✅ Testei performance em casos de alto volume

🔍 Como Testar

  1. GET /api/banks/v1/7 (BNDES): em produção vem "logo_url": null; com este branch vem https://cdn.jsdelivr.net/npm/logos-bancos-br@0/logos/svg/33657248.svg.
  2. tests/banks-logo-url.test.js continua passando (3 testes).
  3. Todas as URLs do arquivo respondem 200 no jsDelivr (testei com HEAD).

📸 Screenshots (se aplicável)

📎 Issues Relacionadas

Relacionado ao #834.

📝 Notas Adicionais

Gerei a partir do data/logo-urls.min.json da 0.8.1 no npm, no mesmo formato do arquivo atual, então o diff só tem as 27 linhas novas.

@vercel

vercel Bot commented Oct 19, 2026

Copy link
Copy Markdown

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T00:09:26.951163Z 150e658 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

1 participant