Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
kaiocpn opened this issue May 21, 2024 · 9 comments · Fixed by #296
Open

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

kaiocpn opened this issue May 21, 2024 · 9 comments · Fixed by #296
Assignees
@TucanoWeb @HbLuca
Labels
bug Something isn't working

Comments

@kaiocpn
Copy link

kaiocpn commented May 21, 2024

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 kaiocpn added the bug Something isn't working label May 21, 2024
@HbLuca HbLuca added this to SOS-RS May 21, 2024
@HbLuca HbLuca moved this to Backlog in SOS-RS May 21, 2024
@alexanderscheibler
Copy link
Contributor

alexanderscheibler commented May 22, 2024 •
edited
Loading

Boa!

@alexanderscheibler alexanderscheibler mentioned this issue May 22, 2024
Closed
TucanoWeb added a commit to TucanoWeb/frontend-sos-rs that referenced this issue May 22, 2024
@TucanoWeb
SOS-RS#287
075f957
@TucanoWeb
Copy link
Contributor

PR: #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
issue_287.mp4

@HbLuca HbLuca linked a pull request May 23, 2024 that will close this issue
#287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos #296
Merged
@HbLuca HbLuca moved this from Backlog to Em desenvolvimento in SOS-RS May 23, 2024
@rhuam rhuam moved this from Em desenvolvimento to Em revisão (dev) in SOS-RS May 24, 2024
larissapissurno pushed a commit that referenced this issue May 25, 2024
@TucanoWeb
#287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicame…
afde606 
…ntos (#296)

* #287

* Delete src/components/Icon directory

* Update SupplyRowInfo.tsx

* RollBack SupplyRowInfo.tsx

* Update SupplyRow.tsx

* Update EditShelterSupply.tsx

* Update CreateSupply.tsx

- De forma a evitar termos genéricos demais, é solicitado ao usuário que registre um recurso com no mínimo 3 caracteres. Validação via Yup.

* Update CreateSupply.tsx

- Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup.

* Update CreateSupply.tsx

* Update CreateSupply.tsx

- Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado.

* Update CreateSupply.tsx

- Bloqueia caracteres especiais;
- Requer no mínimo 3 letras (bloqueia apenas números).
@larissapissurno larissapissurno moved this from Em revisão (dev) to PR Aprovado e Mergeado in SOS-RS May 25, 2024
@larissapissurno
Copy link
Contributor

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

@larissapissurno
Copy link
Contributor

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

@larissapissurno
Copy link
Contributor

adicionado no PR pra staging: #311

@larissapissurno larissapissurno moved this from PR Aprovado e Mergeado to Disponível para QA in SOS-RS May 25, 2024
@larissapissurno larissapissurno moved this from Disponível para QA to PR Aprovado e Mergeado in SOS-RS May 25, 2024
larissapissurno added a commit that referenced this issue May 29, 2024
@larissapissurno @kevinDsousa
@Lmedeiros-leiman @TucanoWeb @xlucaix @rhuam @rodrigooler @diegodario88 @fagundesjg
[release 0.0.4] Merges develop on staging (#311)
a1e8945 
* fix: 🐛 Ajuste do tamanho do botao reload

Fixado tamanho do botão para corrigir erro no Header

* Criado botao

* Adicionado funcoes para o botao. COMENTADO OVERFLOW-X:HIDDEN no global.css

* removendo necessidade de remover overflow-x:hidden no global.css

* resolvendo funcoes

* Finalizando.

durante meus testes esqueci de inverter o estado inicial do botão.

* aplicado o design recomendado pela barbiebrega

* Removido erro de tipagem.

* #287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos (#296)

* #287

* Delete src/components/Icon directory

* Update SupplyRowInfo.tsx

* RollBack SupplyRowInfo.tsx

* Update SupplyRow.tsx

* Update EditShelterSupply.tsx

* Update CreateSupply.tsx

- De forma a evitar termos genéricos demais, é solicitado ao usuário que registre um recurso com no mínimo 3 caracteres. Validação via Yup.

* Update CreateSupply.tsx

- Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup.

* Update CreateSupply.tsx

* Update CreateSupply.tsx

- Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado.

* Update CreateSupply.tsx

- Bloqueia caracteres especiais;
- Requer no mínimo 3 letras (bloqueia apenas números).

* Update - Melhoria na listagem de suplementos (#249)

* feat: add multi option for priority queryParam

* fix(filtro-shelters): add missing typing definition

* fix: name of array of priority to priorities, priority field changed from string to array and removed unused import

---------

Co-authored-by: kevindsousa <kevindiegodasilvasousa@gmail.com>
Co-authored-by: leonardo <lmedeiros045@gmail.com>
Co-authored-by: Leonardo <57924586+Lmedeiros-leiman@users.noreply.github.com>
Co-authored-by: Eric Ricielle <suporte@tucanoweb.com.br>
Co-authored-by: Lucas <ls4388387@gmail.com>
Co-authored-by: Rhuam Sena <rhuam.estevam@doc9.com.br>
Co-authored-by: Diego Dario <diego.dario@gazin.com.br>
Co-authored-by: Rodrigo Oler <roodrigoprogrammer@gmail.com>
Co-authored-by: Diego Dario <25825145+diegodario88@users.noreply.github.com>
Co-authored-by: José Fagundes <fagundesjg@outlook.com>
@larissapissurno larissapissurno moved this from PR Aprovado e Mergeado to Disponível para QA in SOS-RS May 29, 2024
@larissapissurno larissapissurno moved this from Disponível para QA to Em QA in SOS-RS May 29, 2024
@HbLuca
Copy link

HbLuca commented May 29, 2024

@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 HbLuca moved this from Em QA to Disponível Infra in SOS-RS May 29, 2024
@HbLuca HbLuca reopened this May 29, 2024
@github-project-automation github-project-automation bot moved this from Disponível Infra to Backlog in SOS-RS May 29, 2024
@HbLuca HbLuca moved this from Backlog to Disponível Infra in SOS-RS May 29, 2024
@TucanoWeb
Copy link
Contributor

@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
Copy link

HbLuca commented May 29, 2024

@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
Copy link
Contributor

TucanoWeb commented May 29, 2024 •
edited
Loading

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

@rhuam rhuam mentioned this issue May 30, 2024
Merged
rhuam added a commit that referenced this issue May 30, 2024
@rhuam
[release 0.0.4] Merges staging on main (#338)
f2afb66 
* fix: 🐛 Ajuste do tamanho do botao reload

Fixado tamanho do botão para corrigir erro no Header

* Criado botao

* Adicionado funcoes para o botao. COMENTADO OVERFLOW-X:HIDDEN no
global.css

* removendo necessidade de remover overflow-x:hidden no global.css

* resolvendo funcoes

* Finalizando.

durante meus testes esqueci de inverter o estado inicial do botão.

* aplicado o design recomendado pela barbiebrega

* Removido erro de tipagem.

* #287 - [FIX] Itens Cadastrados sem Categoria estão indo para
Medicamentos (#296)

* #287

* Delete src/components/Icon directory

* Update SupplyRowInfo.tsx

* RollBack SupplyRowInfo.tsx

* Update SupplyRow.tsx

* Update EditShelterSupply.tsx

* Update CreateSupply.tsx

- De forma a evitar termos genéricos demais, é solicitado ao usuário que
registre um recurso com no mínimo 3 caracteres. Validação via Yup.

* Update CreateSupply.tsx

- Bloqueia cadastro de items com números e caracteres especiais.
Validação via Yup.

* Update CreateSupply.tsx

* Update CreateSupply.tsx

- Limite de 30 itens retornados enquanto o usuário está digitando o
termo desejado.

* Update CreateSupply.tsx

- Bloqueia caracteres especiais;
- Requer no mínimo 3 letras (bloqueia apenas números).

* Update - Melhoria na listagem de suplementos (#249)

* feat: add multi option for priority queryParam

* fix(filtro-shelters): add missing typing definition

* fix: name of array of priority to priorities, priority field changed
from string to array and removed unused import

---------
AlchemistByte pushed a commit to AlchemistByte/frontend that referenced this issue Jun 6, 2024
@TucanoWeb
SOS-RS#287 - [FIX] Itens Cadastrados sem Categoria estão indo para Me…
140f1c5 
…dicamentos (SOS-RS#296)

* SOS-RS#287

* Delete src/components/Icon directory

* Update SupplyRowInfo.tsx

* RollBack SupplyRowInfo.tsx

* Update SupplyRow.tsx

* Update EditShelterSupply.tsx

* Update CreateSupply.tsx

- De forma a evitar termos genéricos demais, é solicitado ao usuário que registre um recurso com no mínimo 3 caracteres. Validação via Yup.

* Update CreateSupply.tsx

- Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup.

* Update CreateSupply.tsx

* Update CreateSupply.tsx

- Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado.

* Update CreateSupply.tsx

- Bloqueia caracteres especiais;
- Requer no mínimo 3 letras (bloqueia apenas números).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TucanoWeb TucanoWeb

@HbLuca HbLuca

Labels
bug Something isn't working
Projects
SOS-RS
Status: Disponível Infra
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos TucanoWeb/frontend-sos-rs
5 participants
@larissapissurno @alexanderscheibler @kaiocpn @TucanoWeb @HbLuca