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.

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

[MELHORIA/BUG] Remover abrigos desativados #178

Open
HbLuca opened this issue May 15, 2024 · 11 comments
Open

[MELHORIA/BUG] Remover abrigos desativados #178

HbLuca opened this issue May 15, 2024 · 11 comments
Labels
bug Something isn't working

Comments

@HbLuca
Copy link

HbLuca commented May 15, 2024

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

  • Baixa

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

@HbLuca HbLuca added the bug Something isn't working label May 15, 2024
@JuanGustah
Copy link

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

@HbLuca
Copy link
Author

HbLuca commented May 16, 2024

@JuanGustah fechado!
Só tem de observar que o #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
Copy link

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

HbLuca commented May 16, 2024

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

Poderíamos ter os status
LOTADO
DISPONIVEL
DESATIVADO

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

@HbLuca
Copy link
Author

HbLuca commented May 16, 2024

@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 #146, já que tem correlação com este, por se tratar do "abrigo". Além do @JuanGustah que está tratando deste...

@JuanGustah
Copy link

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

JuanGustah commented May 16, 2024

@HbLuca No fim, a correção dependia apenas de uma correção no backend, no qual está descrita no PR SOS-RS/backend#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: #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
Copy link
Author

HbLuca commented May 16, 2024

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

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

HbLuca commented May 16, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants