Closed TucanoWeb closed 4 months ago
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.
@fagundesjg
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
@jlcvp
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.
Parece que foi adicionado um passo a mais para o usuário adicionar um suprimento já cadastrado no banco de dados;
Ao buscar por um suprimento que ainda não foi marcado com uma opção de prioridade e clicá-lo no select, nada é retornado para o usuário, dando a impressão de que o sistema não funcionou.
Parabéns pelo trabalho, @TucanoWeb!
Muito obrigado @evertonpavan
Algumas considerações sobre a feature: remoção de itens não necessários na página de necessidades do abrigo.
- Parece que foi adicionado um passo a mais para o usuário adicionar um suprimento já cadastrado no banco de dados;
- Ao buscar por um suprimento que ainda não foi marcado com uma opção de prioridade e clicá-lo no select, nada é retornado para o usuário, dando a impressão de que o sistema não funcionou.
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 👏 👏 👏
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 ❤️
FIX:
FEATURES:
EXTRA:
Inserção de Icons para utilização em src/components/icons(removed)https://github.com/SOS-RS/frontend/assets/104173294/a447e7a7-fdcf-4c2c-b52d-d89383744c32