frontend icon indicating copy to clipboard operation
frontend copied to clipboard

[BUG] Itens Cadastrados sem Categoria estão indo para Medicamentos.

Open kaiocpn opened this issue 1 year ago • 9 comments

Descrição

Como o campo Categoria no cadastro de um novo item, não esta obrigatório, quando salvamos sem preencher o item vai para a categoria medicamentos.

Problema encontrado

Aqui você dá detalhes sobre o problema, incluindo os passos para reproduzir, se necessário.

Prioridade

  • Baixa: coisas poucas, sistema pode ser usado. Problemas de grafia são incluídos aqui

Solução proposta

Colocar o campo Categoria como obrigatório o preenchimento.

Ambiente

SO: Win11 pro. Navegador: Google Chrome 124.0.6367.208

Evidência

image

kaiocpn avatar May 21 '24 14:05 kaiocpn

Boa!

alexanderscheibler avatar May 22 '24 02:05 alexanderscheibler

PR: https://github.com/SOS-RS/frontend/pull/296

FIX:

  • Ajuste do erro que ocasionava bypass na validação do formulário para categoria;
  • Desabilitação do botão de cadastro se registro existir;

FEATURES:

  • Remoção do item "Não precisa" no campo de prioridade apenas no momento do cadastro do item;
  • Remoção de itens não necessários na página de necessidades do abrigo;
  • Possibilidade de edição do supply ao clicar no registro retornado, se existir, na tela de cadastro.

EXTRA:

  • Inserção de Icons para utilização em src/components/icons

https://github.com/SOS-RS/frontend/assets/104173294/a447e7a7-fdcf-4c2c-b52d-d89383744c32

TucanoWeb avatar May 22 '24 12:05 TucanoWeb

Testado localmente e está funcionando sem erros. PR mergeado em develop, farei o teste de no dev env em seguida

larissapissurno avatar May 25 '24 03:05 larissapissurno

testado em https://dev.sos-rs.com/ e está funcionando corretamente

larissapissurno avatar May 25 '24 03:05 larissapissurno

adicionado no PR pra staging: https://github.com/SOS-RS/frontend/pull/311

larissapissurno avatar May 25 '24 03:05 larissapissurno

@larissapissurno de acordo. Realizado teste com criacao de novo produto "Agenoxi teste" O campo Categoria esta como obrigatorio, caso nao preencha, nao sera salvo.

Vide Loom: https://www.loom.com/share/3fbfbf37fe9c4d2e973ab432957945be?sid=ae4c8b50-baef-4ca7-961b-096c2cbccbf8

HbLuca avatar May 29 '24 17:05 HbLuca

@HbLuca, tudo bem? Espero que sim.

Se esse campo não for obrigatório e o usuário não selecionar corretamente a categoria, qualquer item cadastrado será alocado para medicamento.

A obrigatoriedade da seleção de categoria é justamente para evitar essa desvio.

Se eu tiver entendido incorretamente a issue, peço que me esclareça com mais detalhes por gentileza.

Grande abraço!

TucanoWeb avatar May 29 '24 18:05 TucanoWeb

@TucanoWeb Tudo e contigo?

Pelo que entendi e realizei o teste, ele apenas valida a obrigatoriedade referente ao campo categoria ao tentar adicionar um novo produto, seja ele medicamento ou alimento ou qualquer outra coisa que esteja entre as categorias presentes.

Nao especifica se esta correto ou nao, caso especificasse, o Agenoxi deveria ser verificado e listado como medicamento. A categoria apenas evita o desvio para o medicamento, ja que os produtos estavam indo automaticamente para ele, por nao ter a validacao de obrigatoriedade do campo categoria.

A ti tbm grande abraço!

HbLuca avatar May 29 '24 18:05 HbLuca

@HbLuca, isso mesmo.

Atualmente não existe esse tipo de verificação de categoria. É o usuário que escolhe. O ajuste foi somente para não ir direto para medicamento.

Para que seja possível a certeza de que um item seja categorizado corretamente, é necessário que ele seja previamente validado por um membro da equipe, seja cadastrado previamente ou categorizado através de uma AI.

De qualquer forma, com exceção do primeiro caso (membro da equipe validando), para o restante o campo de categoria se tornará obsoleto.

TucanoWeb avatar May 29 '24 18:05 TucanoWeb