[TREINAMENTO] CRUD de Vacinas — Denilson Rodrigues - #902
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 |
Dario-Arthur
left a comment
There was a problem hiding this comment.
🟠 Problemas encontrados no PR #902
-
Nome duplicado com case diferente não é bloqueado
Ocorre tanto na criação quanto na edição de vacina. Um nome já existente em case diferente (ex: "bcg" quando "BCG" já existe) é aceito nos dois fluxos. Necessário tornar a validação case-insensitive (existsByNameIgnoreCase). -
Nome muito curto (1 caractere) é aceito
Também presente na criação e na edição. O formulário não bloqueia nomes com menos de 2 caracteres antes de enviar. A issue exige 2-100 caracteres — a validação precisa ser aplicada em ambos os fluxos.
O que precisa ser feito
1 - Ajustar a validação de nome duplicado no backend para ignorar maiúsculas/minúsculas — na criação e na edição
2 - Adicionar validação de tamanho mínimo (2 caracteres) no formulário e/ou reforçar no backend — na criação e na edição
Build (./mvnw compile, tsc e npm run build) passando sem erros. Demais pontos testados (botão adicionar, editar/excluir sem erro 500, bloqueio de exclusão com mensagem, busca, pré-preenchimento, 404, mobile) estão funcionando corretamente.
… Valid no controller de vacinas #887
|
Corrigidos os dois problemas apontados na revisao:
Testado localmente: nome duplicado com case diferente bloqueado na criacao e na edicao, nome de 1 caractere bloqueado, build do frontend e backend sem erros, demais fluxos continuam funcionando normalmente. |
O que mudou?
Implementado CRUD completo de vacinas (criacao, edicao e exclusao), incluindo validacao de nome obrigatorio (2 a 100 caracteres), bloqueio de nome duplicado e bloqueio de exclusao de vacina vinculada a paciente.
Tarefas Relacionadas
Mudancas Realizadas
Evidencias
Video gravado demonstrando: criar vacina, editar vacina, bloqueio de nome duplicado com mensagem correta, exclusao de vacina sem vinculo, busca por nome na listagem.
Video.Project.mp4