-
Notifications
You must be signed in to change notification settings - Fork 328
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.
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
#287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos #296
Conversation
Já utilizamos uma biblioteca de ícones 'Lucide React'. Não precisa nem deverá ser utilizados ícones de outras fontes/pacotes, por motivos de padronização. |
Obrigado pelo retorno. Removi o diretório de ícones que havia colocado anteriormente e ajustei o componente incluindo um ícone da biblioteca já utilizada. |
se possível, atualiza o título do PR pra o título do issue |
Done ✅ |
Parabéns pelo trabalho, @TucanoWeb! Algumas considerações sobre a feature: remoção de itens não necessários na página de necessidades do abrigo.
|
Muito obrigado @evertonpavan
Suas considerações são válidas e os ajustes foram realizados. Muito obrigado pelo feedback. 🚀 PS: Importante destacar que o belo trabalho do auto-complete foi feito pela @larissapissurno 👏 👏 👏 |
Após atualização: issue_287_updated.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parabéns pelo trabalho @TucanoWeb 👏 ! no geral o código parece bom, mas fiz apenas uma revisão geral pois fiquei pensando aqui sobre uns pontos:
- Itens com apenas duas letras, como "pá", não sao validados pois existe uma limitação no filtro
- Acho que pode ser um pouco problemático a gente desabilitar o botão de salvar caso existam nomes de itens que contém a string, pois podem existir casos onde o item cadastrado é novo, mas está contido em algum outro nome de item existente, ex: (Novo item: carro, Item existente: carroça)
- Por fim, achei um pouco contra-intuitivo a edição de itens na tela de cadastro de novo item, talvez podemos mudar para um fluxo mais linear, como:
- usuario digita o nome do item normalmente no campo de cadastro
- ao clicar em salvar, o sistema valida se já existe itens que contém o nome digitado
- se houver, mostra uma modal de confirmação, informando que existem itens com nome similar já cadastrado (e talvez aqui permitir a edição? nao sei), na confirmação pode ter um botão "Cadastrar novo item" e um "Cancelar", nesse caso podemos até focar o "Cancelar" pra não induzir o usuario a só dar "enter" e sempre cadastrar um novo.
vou trazer essa conversa pro tópico existente pra pegarmos a opinião do pessoal de produto tbm o/
topico: https://discord.com/channels/1237553809371299950/1242105427697729586
@larissapissurno , muito obrigado pelo feedback. 1 - Respondi ele mais acima; 2 - Acredito que o trabalho do usuário de escoher um sinônimo gera menos transtorno e "bagunça" que ele cadastrar um item duplicado ou genérico, do tipo "carro". Nesse caso, ele colocaria algo mais específico, que ao meu ver, é o ideal, como "Carro de mão", "Carrinho de mão", "Automóvel com Caçamba"... etc; 3 - Essa validação, ao meu ver, deve ocorrer no início do fluxo, ao invés de ocorrer no final. Assim que o usuário digita o termo, as similaridades são impressas, seguindo um fluxo que, ao meu ver, é mais linear e segue a regra de negócio proposta, mas essa é minha opinião. Mais uma vez, muito obrigado pelo feedback ❤️ |
- 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.
- Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup.
- Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado.
- Bloqueia caracteres especiais; - Requer no mínimo 3 letras (bloqueia apenas números).
* 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>
* 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 ---------
…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).
FIX:
FEATURES:
EXTRA:
Inserção de Icons para utilização em src/components/icons(removed)issue_287.mp4