-
Notifications
You must be signed in to change notification settings - Fork 319
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
Evitar duplicatas no cadastro de novos itens (usando autocomplete na busca) #254
Comments
opa, posso pegar essa @giggio? |
@larissapissurno Pode, sim! Se precisar de apoio fala com a galera no Discord. Esse item tem prioridade alta, então tem dar uma corrida! Bora! |
on it! tranquilo, to acompanhando o discord lá, qualquer coisa dou um grito o/ |
pronto pra code review, @giggio, poderia mover pra coluna de revisão? thx! |
@giggio qq, temos algum SLA por prioridade de tasks/bugs? seria legal uma definição nesse sentido, o que acha? |
Movido! |
Opa, bom dia. Veja se a solução abaixo atende:
|
@TucanoWeb pelo visto temos implementações para resolver problemas parecidos. Criei um tópico no Discord pra discutir como atuar: https://discord.com/channels/1237553809371299950/1242105427697729586 |
O PR #275 Satisfaz as necessidades de produto para essa issue. |
## Related Issue Closes #254 ## Overall Este PR tem como objetivo alterar a forma como é feita a busca de itens na tela de abrigo para evitar a duplicação de itens. As alterações propostas incluem: - Remove o botão de cadastrar novo item no topo da página - Adiciona autocomplete para busca de itens - Adiciona opção de cadastrar novo item no autocomplete ## Screen recording https://github.com/SOS-RS/frontend/assets/8760873/74db3fa0-c45c-4b7a-afb2-d3d5279c0106
@SOS-RS/qas poderiam confirmar se essa issue foi testada em QA? |
@larissapissurno testando aqui... |
@HbLuca os itens estavam sendo movidos automaticamente pra done quando o PR mergeava, vou mover todos que estao em done e sem comment de QA pra coluna "Disponível pra QA", tudo bem? |
@larissapissurno 1 - Ficará a cargo/boa fé do povo? Já que vi que o cenário 1 foi o único que atendeu no ambiente de Q.A, Prod e Dev, cenário 2 e 3 não foram contemplados. 2 - Não existe aviso sobre duplicação de item, apenas o que ocorre é caso adicione o item com modificação de prioridade e quantidade (Não sei se isso seria algo agravante, lembrando que isso é continuidade do 1), ele deixa o valor atual e nem a mensagem de erro sobre item não encontrado é apresentado. 3 - Ao pesquisar, por exemplo bombona de água que está em Eletrodomésticos e Eletrônicos e adiciona-lo em outro local como Equipamentos de Emergência, ele está sendo aceito. Local realizado: Igreja Lugar de adoração. Vide Loom: |
@larissapissurno sem problemas, pode mover. |
oi @HbLuca, obrigada pelo rápido retorno 🚀
não sei se entendi corretamente, mas se estiver se referindo ao fato de continuar deixando qualquer pessoa adicionar o item, eu concordo com vc em partes, por um lado eu penso que deixar qualquer pessoa cadastrar qualquer coisa é um pouco arriscado, mas por outro, se deixarmos essa responsabilidade para o lado do sistema, isso pode causar um gargalo no sentido de termos que gerenciar isso via banco de dados e talvez criar uma interface pro usuário solicitar uma adição de item talvez? No mais acho bem válido seu ponto, pra mim faz sentido trazermos isso pro time de produto pensar junto com a gente, o que acha?
ótimo ponto, temos uma explicação pra isso. Se observar o PR, vai ver que temos um possível PR duplicado, acontece que o PR duplicado implementa apenas a validação no cadastro, enquanto o meu PR implementa o autocomplete, no tópico que estou linkando abaixo concordamos em seguir com os dois PRs, porém o outro não foi mergeado ainda, acho que podemos linkar ele nessa issue tbm e colocar ela on hold até termos o outro PR enviado para QA, o que acha? https://discord.com/channels/1237553809371299950/1242105427697729586 PR de validação no cadastro: #229
como mencionado acima, o PR #229 deve corrigir este problema |
1 - Sim, referente as pessoas mesmo. Faz sentido o parcial, podemos ver sim, mas acredito que seja a mesma tratativa/resposta referente a cadastro de locais, que seguirá aberto. 2/3 - Ótimo, o de autocomplete está de acordo, ao digitar algumas letras ele já faz o serviço. Referente ao item, é um pouco mais delicado, já que ele pode ser algo de necessidade urgente e ai alguém colocar como apenas necessário ou até modificar quantidade, se tiver controle de log, BD "vai cantar"... O 229 é interessante pelo aviso, mas deve ver de bloquear e não deixar ir até o pop-up. |
@larissapissurno e @HbLuca, boa noite. Como estão? Recebi a notificação de referência à PR. A que inclui a validação e a limitação na página de cadastro é a #296 Importante destacar que a ideia de agregar as soluções já havia sido proposta, brilhantemente, pela @larissapissurno. |
@HbLuca entendo, sera entao que poderiamos deixar mais claro que ele está editando um item existente? lembrando que hoje já é possivel alterar essas informações ali na lista de items para o abrigo o que acha @TucanoWeb, teria alguma outra sugestão? |
Boa noite @TucanoWeb estou bem e tu? Hmmm... Com a 296 (a mergear) e 275 (mergeado) já teriamos o autocomplete e também a "trava" referente a duplicação. @larissapissurno no 296 passado pelo @TucanoWeb acredito que já ocorra isso de deixar claro sobre item existente (ao menos vendo os vídeos compreendi isso). Parabéns aos 2. |
mergeei o PR do @TucanoWeb, vou add no PR de release pra staging |
temos esse bug que pode estar relacionado ao autocomplete: #298 |
@larissapissurno, sim... está mesmo! |
@larissapissurno só temos um porém... Possívelmente isso que pode acarretar na parte de cadastro de itens de aparecer duplicado, conforme imagem abaixo: |
@HbLuca, bom dia. Essa redundância ocorre devido aos registros retornados do banco de dados. Essa normalização vai ocorrer pelo DB, backend ou esse controle vai acontecer somente no front? Se não for via front, acredito que as issues relacionadas podem ser fechadas e uma nova aberta no back. Via DB, na minha opinião, é:
|
@TucanoWeb não poderia modificar esta issue para FULL? |
Boa noite @HbLuca. Acredito que somente no back já seja o suficiente. Se vocês concordarem, pode abrir a Issue no back e me dar um toque que eu faço esse ajuste. |
Eu concordo que podemos tratar essa demanda de back em uma nova issue, assim podemos seguir em frente com essa task e podemos dar andamento com a outra independentemente. |
@HbLuca os ajustes pendentes já estão todos em staging, poderia verificar novamente? 🙏 |
@larissapissurno vou verificar e ja criei um em back, @TucanoWeb caso queira ver... |
@larissapissurno feito. |
TL;DR
Mudança no cadastro de novos itens para evitar duplicatas, otimizando a quantificação das demandas. O botão de cadastro de novos itens deixa de existir, a busca vira o meio para cadastrar itens que ainda não existem.
Problema
🗣️ **User story:** Eu sou um doador (CD ou pessoa física). Eu estou tentando entender a necessidade de itens de um abrigo. Mas existem muitos itens similares cadastrados como “água” e “água 1L” porque as pessoas não têm consciência do que já existe o que faz com que eu não consiga entender a demanda de forma clara e objetiva.Precisamos garantir que o registro seja eficiente para o usuário, entregando a informação centralizada. Por isso, é importante que o usuário possa ver quais itens já foram cadastrados e evite cadastrar itens muito similares.
Design
https://www.figma.com/design/uy6ukD6inEdpvaKVhIyobX/SOS-RS?node-id=1002%3A1052&t=gMoiIFBjpx9R1EX3-1
Critérios para aceitação
Critério de sucesso
Centralização dos itens em uma lista coesa no banco de dados a partir da qual o usuário pode identificar as necessidades e a plataforma pode quantificar as demandas.
The text was updated successfully, but these errors were encountered: