Skip to content

fix: corrige ordenação de NCMs na tabela de edição#22

Open
mariana-lins wants to merge 1 commit intodevelopfrom
fix/ncm-ordering
Open

fix: corrige ordenação de NCMs na tabela de edição#22
mariana-lins wants to merge 1 commit intodevelopfrom
fix/ncm-ordering

Conversation

@mariana-lins
Copy link
Contributor

Corrige ordenação incorreta de NCMs na tabela de edição

Tipo de Pull Request:

  • Correção de Bug (Bugfix)

Referência da Task

Jira/Ticket: Relatado por desenvolvedor Victor - NCMs aparecendo em ordem incorreta na tabela

Descrição das Alterações - Changelog

  • Refatorada estrutura de dados de Array<Array<NCM>> para Array<NCM> no useEffect que processa NCMs
  • Removida camada extra de arrays que causava desalinhamento entre índice do produto e NCM correspondente
  • Implementado acesso direto orderNcms[index] ao invés de orderNcms[index][0]
  • Adicionado optional chaining (?.) para null safety ao acessar filhos de NCM
  • Garantido mapeamento 1:1 entre posição do produto e NCM exibido conforme ordem enviada pelo backend

Instruções de Teste

Passos para testar:

  1. Fazer upload de um PDF com múltiplos produtos
  2. Aguardar processamento completo
  3. Acessar a página de edição da tabela (TableEdit)
  4. Verificar se os NCMs exibidos correspondem aos produtos corretos
  5. Comparar com os dados retornados pelo backend na aba Network do DevTools

Resultado esperado:

  • Cada produto deve exibir seu NCM correto na ordem enviada pelo backend
  • O primeiro NCM do backend deve aparecer no primeiro produto do frontend
  • Não deve haver repetição ou embaralhamento de NCMs entre produtos diferentes

Reviewers Recomendados

  1. @[desenvolvedor-backend] - Motivo: Validar se a estrutura de dados recebida do backend está sendo corretamente interpretada
  2. @ victorrgodoy - Motivo: Confirmar que o bug relatado foi corrigido
  3. @[tech-lead] - Motivo: Revisar simplificação da estrutura de dados e melhorias de código

Observações

  • Bug identificado: estrutura de dados criava array bidimensional onde todos os índices referenciavam o mesmo conjunto de NCMs
  • Solução aplicada: transformação direta para array flat, eliminando complexidade desnecessária
  • Melhoria adicional: adicionado null safety com optional chaining para prevenir erros em casos edge
  • Impacto: baixo risco, mudança isolada no processamento de NCMs sem afetar outras funcionalidades

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