Skip to content

Resolução issue #48 + Ajustes modal e regra de remoção de gerente#55

Merged
Artur-Falavinha merged 3 commits intomainfrom
feat-Navbar-dinamica-perfil-usuario
Apr 6, 2026
Merged

Resolução issue #48 + Ajustes modal e regra de remoção de gerente#55
Artur-Falavinha merged 3 commits intomainfrom
feat-Navbar-dinamica-perfil-usuario

Conversation

@xXbrasilXx
Copy link
Copy Markdown
Collaborator

@xXbrasilXx xXbrasilXx commented Mar 30, 2026

📝 Descrição

Implementação da issue #48 juntamente com ajustes no design do modal de remoção de gerente e bloqueando a opção de remover o último gerente. + Vínculo cliente e gerente por CPF

Principais alterações:

  • Ajuste da navbar para cada perfil do sistema
  • Feito redirecionamento pra "nova" home de gerente, agora é assim:
image - Ajuste na forma de exibir o erro ao tentar excluir último gerente - Feito o vínculo de clientes com gerentes através de CPF ao invés de nome.

🛠 Tipo de Mudança

  • FEAT: Nova funcionalidade.
  • 🐛 FIX: Nova funcionalidade.
    (os dois né)

🧪 Guia de Testes

Como reproduzir:

  1. Certifique-se de que a massa de dados (mock/*.json) está atualizada com os arquivos enviados nesta PR.
  2. Inicie o mock server rodando o comando node mock-server.js (rodando na porta 3000).
  3. Inicie a aplicação Angular rodando o comando ng serve.
  4. Acesse a aplicação com cada tipo de usuário: CLIENTE, ADMINISTRADOR, GERENTE
  5. Verifique se em cada perfil, a navbar exibe somente as páginas liberadas pro usuário logado. (Referências nas imagens)
  6. Até agora é referente à issue Tornar Navbar Dinâmica com base no Perfil de Usuário #48 , agora com relação à outra parte deste PR, se trata de um problema de outro PR (Edu sepa q vai ver).
  7. Nessa outra parte, basta tentar remover um gerente pra ver o modal novo
  8. E tentar excluir o último gerente, que o sistema não vai permitir

Resultado esperado:

  • A navbar deve estar correta e exibir as rotas certas e redirecionar pra página específica sem quebrar.
  • ***ATENÇÃO: Para o perfil de admin a rota home não existe, por isso NÃO é possível clicar nela (não é um erro).
  • Não deve permitir excluir o último gerente e não quebrar a tela, basicamente deve mostrar um toast.

📸 Screenshots

Navbar Admin:
image

Navbar Gerente:
image

Navbar Cliente:
image

Novo modal de exclusão de gerente:
image

Erro ao tentar remover o último gerente:
image

@xXbrasilXx
Copy link
Copy Markdown
Collaborator Author

Diversos ajustes realizados para funcionamento da aplicação utilizando o cpf do gerente para relacionar o cliente (de forma padronizada com o managerDocument. Além disso, ajustado alguns CPFs que não eram válidos e davam erro ao consultar a conta.
Abaixo tem um print dos principais arquivos alterados e que merecem atenção, pois acabei fazendo o commit de merge down (com diversas alterações gerais) junto com os ajustes de conflito.
image

Copy link
Copy Markdown
Collaborator

@Artur-Falavinha Artur-Falavinha left a comment

Choose a reason for hiding this comment

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

Revisei a PR na branch atual, cobrindo build, navegação por perfil e fluxo visual de remoção de gerente.

Validação executada:

  1. Build do frontend
  2. Login com cliente para validar navbar dinâmica
  3. Login com gerente para validar navbar dinâmica
  4. Login com administrador para validar navbar dinâmica
  5. Abertura do modal de remoção de gerente
  6. Revisão do código alterado no mock-server.js e nos componentes impactados

Teste visual

  1. Navbar do cliente
Image

Status: certo
Motivo: a navbar do cliente renderiza apenas as opções do autoatendimento e a navegação principal aparece separada por perfil, como esperado no escopo da issue #48.

  1. Navbar do gerente
Image

Status: certo
Motivo: a navbar do gerente renderiza apenas opções do módulo de gerente, sem exibir links do cliente ou do administrador.

  1. Navbar do administrador
Image

Status: certo
Motivo: a navbar do administrador renderiza as opções de backoffice, com navegação distinta dos outros perfis.

  1. Modal de remoção de gerente
Image

Status: certo
Motivo: o modal novo abre corretamente, comunica a ação crítica e visualmente está consistente com o fluxo esperado de confirmação.

Conclusão da validação visual

  • A parte visual principal da PR está funcionando bem.
  • A navbar dinâmica por perfil foi validada com sucesso.
  • O modal novo de remoção de gerente também está funcionando visualmente.
  • O problema restante da PR não está na UI principal, e sim na consistência funcional da exclusão no mock/backend local.

Review de código

  1. [P1] Exclusão de gerente não remove a credencial de login
    Arquivo: front-end/mock-server.js
    Trecho afetado: rota DELETE /admin/gerentes/:cpf

Problema:

  • A nova implementação da exclusão passou a carregar e salvar apenas gerentes.json e conta-banco.json
  • O fluxo deixou de atualizar auth.json
  • Com isso, o gerente é removido da listagem e as contas são realocadas, mas a credencial do mesmo usuário continua existente no mock de autenticação

Impacto:

  • O CRUD de gerentes fica inconsistente
  • Um gerente excluído pode continuar autenticando no sistema
  • Isso reabre acesso para um perfil que o administrador acabou de remover

Conclusão final da review

  • Visualmente, a PR está em bom estado e atende a maior parte do objetivo da issue #48
  • Porém, do ponto de vista funcional e de integridade do fluxo administrativo, ainda há um problema impeditivo
  • Vou aprovar para não travar o fluxo do merge, daí eu resolvo em outra PR esses problemas que encontre.

@Artur-Falavinha Artur-Falavinha merged commit 0b850f7 into main Apr 6, 2026
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