search

SOS-RS / frontend

Auxílio RS: Projetos de Resposta a Emergências por Chuvas e Alagamentos
https://sos-rs.com
MIT License
596 stars 320 forks source link

[MELHORIA/BUG] Remover abrigos desativados #178

Open HbLuca opened 6 months ago

HbLuca commented 6 months ago

Descrição

Ao verificar no site existem alguns locais que estão desativados.

Problema encontrado

Ao realizar a busca por abrigo ou endereço, digitando desativado, encontrei alguns locais desativados.

Prioridade

Soluções propostas

1 - Retirada/remoção dos abrigos que estão desativados, caso a informação esteja correta e foi passada por fonte confiável. 2 - Criar um filtro de locais desativados. Caso opte por isto, deverá modificar o cabeçalho de Abrigos disponíveis para Abrigos desativados.

Ambiente

Desktop, Windows 10 - Microsoft Edge.

Evidência

image

image

JuanGustah commented 6 months ago

@HbLuca Acredito que para esse problema, a segunda abordagem faz mais sentido. Vou trabalhar em cima disso.

HbLuca commented 6 months ago

@JuanGustah fechado! Só tem de observar que o https://github.com/SOS-RS/frontend/issues/146 não foi contemplado com a modificação de cabeçalho. Ao realizar a busca de Abrigo indisponível e sem informação de disponibilidade não é modificado o Abrigo Disponível (qtde em parênteses), conforme imagem abaixo. Talvez tenha de verificar com o pessoal de Design/UX, para tal.

image

JuanGustah commented 6 months ago

@HbLuca Perfeito, irei falar com a equipe de design sim! Comecei a desenvolver e percebi que a listagem do campo de status do abrigo tem a ver com a relação da quantidade de vagas disponíveis ( caso tenha vagas está disponível, indisponível do contrário, e caso não seja informado a quantidade de vagas é sem informação de disponibilidade), o que é uma informação diferente da que estamos tratando aqui (um abrigo ser desativado) e que seria interessante manter da forma que está. No meu ver, seria interessante tratarmos esses abrigos desativados como um soft-delete (não exibir mais o registro no sistema mas não remover o registro no banco de dados por questões de auditoria/histórico), mas acho que vale a discussão.

HbLuca commented 6 months ago

@JuanGustah de acordo. Caso necessário, me chame para conversarmos sobre, com a equipe de design.

Totalmente de acordo referente ao soft-delete, só tem de ver com quem cuida da parte de servidor, para entender melhor sobre o espaço/qtde de dados, sim vale.

HelioDantas commented 6 months ago

Poderíamos ter os status LOTADO DISPONIVEL DESATIVADO

O status de lotado e disponível poderiam está atrelado a quantidade de vagas

HbLuca commented 6 months ago

@HelioDantas observando o projeto no Figma, está assim no escopo inicial:

image

Seria um pouco diferente, mas assim, caso queira alterar a parte de status, teria de contactar a equipe de Design/UX para verem se estão de acordo sobre.

E isso de status, teria de verificar também com o @kevinpiske, que está tratando o https://github.com/SOS-RS/frontend/issues/146, já que tem correlação com este, por se tratar do "abrigo". Além do @JuanGustah que está tratando deste...

JuanGustah commented 6 months ago

@HelioDantas Era essa linha de racicínio que eu estava seguindo inicialmente. Mas ao meu ver, não é interessante juntar as duas coisas porque pode confundir o usuário. Acredito que dá as duas opções para ele escolher ajuda mais para esse contexto.

JuanGustah commented 6 months ago

@HbLuca No fim, a correção dependia apenas de uma correção no backend, no qual está descrita no PR https://github.com/SOS-RS/backend/pull/125, que insere a filtragem automática dos abrigos desativados. Mas talvez haja a necessidade de inserir um filtro para exibir esses abrigos desativados, por isso adicionei também uma opção de filtragem nesse mesmo PR, e fiz uma interface para ele no PR do front-end: https://github.com/SOS-RS/frontend/pull/204. Ainda sim, o trabalho ainda não acaba aqui. Ainda é preciso corrigir esses registros indicados nos vídeo com a flag no banco de prod (no qual não tenho acesso). Além de que poderia ser interessante adicionar um botão para sinalizar que o abrigo está desativado, o qual poderia gerar uma nova issue.

HbLuca commented 6 months ago

@JuanGustah ótimo trabalho! Puxou tanto a parte front quanto back... Esperar subir em prod para testar.

Neste caso, o botão seria como o click das opções dos abrigos ou como o filtrar resultados? Referente ao banco, acredito que conversando com o pessoal de infra ou adm consigas.

JuanGustah commented 6 months ago

@HbLuca Muito obrigado!

O botão que sugeri seria mais próximo ao click das opções dos abrigos sim. Talvez uma bandeira literalmente para sinalizar que aquele abrigo está desativado. Ou podemos deixar junto a seção de editar abrigo.

De toda forma é preciso verificar o fluxo que isso gera. Podemos deixar a sinalização livre para qualquer usuário alterar, como está os campos de quantidades de pessoas abrigadas e aceita animais, porém essa edição livre para qualquer usuário está sendo repensada para um fluxo por usuários. Portanto é preciso analisar como essa feature nova se encaixa nesse fluxo.

HbLuca commented 6 months ago

@JuanGustah de nada.

Interessante a ideia...

Neste caso, tem de ser realizada a análise junto com o pessoal de Design/UX, pois impactaria no Figma deles, da mesma forma que sugestionei no 146.