SOS-RS / frontend

Auxílio RS: Projetos de Resposta a Emergências por Chuvas e Alagamentos

https://sos-rs.com

MIT License

596 stars 322 forks source link

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

Open kaiocpn opened 5 months ago

kaiocpn commented 5 months ago

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

alexanderscheibler commented 5 months ago

Boa!

TucanoWeb commented 5 months ago

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

larissapissurno commented 5 months ago

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

larissapissurno commented 5 months ago

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

larissapissurno commented 5 months ago

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

HbLuca commented 5 months ago

@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

TucanoWeb commented 5 months ago

@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!

HbLuca commented 5 months ago

@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!

TucanoWeb commented 5 months ago

@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.