Skip to content

hotfix: corrige contagem de linhas válidas na prévia de importação de usuários - #542

Merged
BlMedeiros merged 2 commits into
mainfrom
fix/importacao-planilha-confirmar-desabilitado
Jul 24, 2026
Merged

hotfix: corrige contagem de linhas válidas na prévia de importação de usuários#542
BlMedeiros merged 2 commits into
mainfrom
fix/importacao-planilha-confirmar-desabilitado

Conversation

@BlMedeiros

Copy link
Copy Markdown
Contributor

Contexto

Reportado que a importação de usuários por planilha (#441) sempre dizia "linha válida" na prévia, mas nenhum usuário era criado. Reproduzido localmente: no modo prévia (dryRun), o backend retorna created/skipped zerados de propósito (nada é persistido ainda), mas o modal usava esse mesmo campo pra decidir se o botão "Confirmar importação" ficava habilitado — resultado: o botão ficava permanentemente desabilitado depois de qualquer "Analisar planilha", mesmo com todas as linhas válidas.

O que muda

ImportUsersModal passa a contar linhas válidas/ignoradas/com erro a partir de result.rows (row.status), em vez dos totais created/skipped do topo da resposta — que só refletem o que foi de fato persistido e por isso vêm zerados em dryRun. O banner de "Importação concluída" continua usando result.created/result.skipped, já que ali eles são reais.

Issue relacionada

Sem issue aberta — bug foi reportado direto em conversa, sem ticket no board.

Como testar

  1. Logar como usuário com permissão de criar usuários e ir em /usuarios/novo.
  2. Clicar em "Importar usuários por planilha" e escolher um CSV (name,email) com pelo menos uma linha válida.
  3. Clicar em "Analisar planilha" — antes do fix, o banner mostrava "0 linha(s) válida(s)" e o botão "Confirmar importação" ficava cinza/desabilitado mesmo com linhas válidas nas mensagens (Lx Linha válida.). Depois do fix, o banner mostra a contagem correta e o botão habilita.
  4. Clicar em "Confirmar importação" e confirmar que os usuários aparecem na listagem (Importação concluída: N criado(s)).

Tipo de mudança

  • Nova feature
  • Correção de bug
  • Refatoração (sem mudança de comportamento)
  • Documentação
  • Infraestrutura / config / build
  • Outro: ___

Checklist do autor

  • Código segue convenções definidas em CONTRIBUTING.md
  • Validei localmente que a aplicação compila/gera build sem erros (quando aplicável) — next build compila, tipa e gera as 56 páginas estáticas; a falha de EPERM ao final é um symlink do output standalone no Windows local, pré-existente e não relacionado a esta mudança.
  • Verifiquei que não há erros de análise estática ou alertas relevantes no código (quando aplicável) — pnpm --filter @portal/core lint limpo.
  • Confirmei que não há erros de tipagem/TypeScript no escopo da mudança (quando aplicável) — tsc --noEmit limpo em packages/core.
  • Testei manualmente os cenários principais — reproduzido o bug e validada a correção ponta a ponta em localhost:3006 (prévia + confirmação real criando usuários).
  • Componentes novos/alterados documentados no Storybook (se aplicável) — não é componente do design system (packages/core, não packages/ui).
  • Documentação atualizada (se aplicável)
  • Não introduzi dependências novas sem alinhamento prévio

Notas pro revisor

Não há teste automatizado cobrindo isso: ImportUsersModal é Client Component sem infra de teste de componente no monorepo ainda (mesma dívida do ProfileMenu, registrada no AGENTS.md — falta jsdom/@testing-library/react).

@BlMedeiros BlMedeiros added the bug Algo não está funcionando label Jul 24, 2026
@BlMedeiros BlMedeiros self-assigned this Jul 24, 2026
@PabloTzeliks
PabloTzeliks self-requested a review July 24, 2026 17:44
@BlMedeiros
BlMedeiros changed the base branch from develop to main July 24, 2026 17:44
@BlMedeiros BlMedeiros changed the title fix: corrige contagem de linhas válidas na prévia de importação de usuários hotfix: corrige contagem de linhas válidas na prévia de importação de usuários Jul 24, 2026
@BlMedeiros
BlMedeiros merged commit 5654009 into main Jul 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Algo não está funcionando

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants