Skip to content

fix: corrigindo lista de distribuidoras e ucs

Adne Moretti requested to merge 172/lista-distribuidoras into develop

Descrição

Modificações nos seguintes arquivos:

  • src/pages/distribuidoras/index.tsx;
  • src/pages/uc/index.tsx;
  • src/pages/distribuidoras/distributorId;

Motivação e contexto

Após investigação, a página estava redirecionando para url com id de distribuidor igual a 1, pois os dados da requisição não haviam sido carregados ainda e existir condicional que possibilitava redirecionamento para -1 quando a lista de distribuidoras estava undefined;

Adicionando condicional para que não haja redirecionamento quando ainda não existe o dado, somente quando já existe a lista de distribuidoras, mesmo que vazia.

Issue relacionada

172-lista-de-distribuidoras-nao-carrega-corretamente

Principais implementações

  • Condicional para resolver problema na lista de distribuidoras;
  • Condicional para resolver problema na lista de unidades consumidoras;
  • Adicionando página de distribuidora vazia assim como padrão em unidade consumidora vazia;

Como isso foi testado?

Testado no browser;

Tipos de mudança

Marque com um "x" as caixas correspondentes:

  • Correção de bug (não-interrompção de mudança que corrige um problema)
  • Nova funcionalidade (mudança que adiciona funcionalidade)
  • Mudança de quebra (alteração que interrompe o comportamento anterior)
  • Melhoria de desempenho
  • Mudança de refatoração (alteração que não adiciona nova funcionalidade nem corrige bugs)
  • Outra mudança (descreva abaixo):

Checklist

Marque com um "x" as caixas correspondentes:

  • O código segue as diretrizes de estilo deste projeto
  • Foram adicionados testes para cobrir as novas funcionalidades ou mudanças
  • Todos os testes passam
  • As alterações foram revisadas por pelo menos um colega de equipe
  • A documentação foi atualizada, se necessário
  • As alterações não quebram o fluxo de trabalho de nenhum outro usuário ou sistema

Notas adicionais

As seguintes linhas de código foram retiradas. Verificar utilização, pois acredito que não estão sendo utilizadas no sistema, apenas fazendo requisições repetidas;

`` const activeDistributorId = useSelector(selectActiveDistributorId);

if (activeDistributorId) { router.push(/distribuidoras/${activeDistributorId}); }

const activeConsumerUnitId = useSelector(selectActiveConsumerUnitId);

if (activeConsumerUnitId) { router.push(/uc/${activeConsumerUnitId}); } ``

Edited by Adne Moretti

Merge request reports