[TREINAMENTO] CRUD de Vacinas — Paulo Moura Menezes - #884
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Testei o PR localmente e ficou tudo certo:
npx tsc --noEmit e npm run build passando sem erros
CRUD completo de vacinas (criar, editar, excluir) funcionando
Validação de nome duplicado (case-insensitive) funcionando
Vínculo vacina ↔ paciente testado: ao vincular, a exclusão fica bloqueada corretamente
Responsividade ok
Observação à parte, sem relação com esse PR: encontrei um erro (PUT 400 / GET 500) na tela de Registro Anual do paciente, mas é em annual-registry.api.ts, fora do escopo de vacinas — vou reportar separadamente.
Aprovado 👍
Dario-Arthur
left a comment
There was a problem hiding this comment.
Complementando minha review anterior: ao validar mais a fundo contra os critérios de aceite da issue #869, encontrei um ponto que precisa de ajuste antes do merge.
O bloqueio de exclusão de vacina vinculada a paciente funciona (ícone de cadeado aparece corretamente), mas não há nenhuma mensagem explicando o motivo do bloqueio. Testei hover e não aparece tooltip nem texto. A issue exige explicitamente: "A interface informa o motivo quando a exclusão estiver bloqueada."
Sugestão de ajuste: adicionar um title/tooltip no ícone do cadeado, ou um texto ao lado (ex: "Vacina em uso por paciente").
O restante da entrega segue validado build, CRUD completo, validação de nome duplicado, e o próprio bloqueio de exclusão funcionando na prática.
O que mudou?
Resumo das mudanças de frontend e backend implementadas: criação, edição e exclusão de vacinas; bloqueio de exclusão quando vinculada a paciente; validação de nome duplicado. Não foi possível realizar o teste do 'Bloqueio de exclusão' pois não havia paciente cadastrado, conforme imagem anexada.
Tarefas Relacionadas
Closes #881
Como testar
Checklist
Evidências