Skip to content

(Sugestão) Correção do título com/sem filtros #247

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

Open
llayse opened this issue May 18, 2024 · 8 comments · Fixed by #270
Open

(Sugestão) Correção do título com/sem filtros #247

llayse opened this issue May 18, 2024 · 8 comments · Fixed by #270
Labels
enhancement New feature or request

Comments

@llayse
Copy link

llayse commented May 18, 2024

Descrição

O título do site "Abrigos Disponíveis(xx)" não corresponde a quantidade de abrigos disponíveis, mas sim abrigos cadastrados. (sem filtro aplicado)

Problema encontrado

Se utilizarmos, o filtro Status do Abrigo, selecionando a checkbox:

  • Abrigos Disponíveis: ok, faz sentido o texto corresponder a 'Abrigos Disponíveis(número)'
  • Abrigos Indisponíveis: não faz sentido o título ser "Abrigos disponíveis(número)".
    image

Solução proposta

Quando não aplicado filtros, o título poderia ser "Abrigos cadastrados" ou "Abrigos adicionados" ou "Total de Abrigos".
Quando aplicado filtros, o título poderia ser "Abrigos encontrados(número)" e talvez até exibir quais filtros estão sendo utilizados. (Atualmente, para ter ciência disso, é preciso dar um click no botão de filtros para o usuário "se lembrar" do que filtrou).

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

HbLuca commented May 18, 2024

Correlacionado ao #146

@HbLuca HbLuca added this to SOS-RS May 19, 2024
@HbLuca HbLuca added enhancement New feature or request and removed bug Something isn't working labels May 19, 2024
Tiago-Silva added a commit to Tiago-Silva/SOS-RS-frontend that referenced this issue May 19, 2024
@HbLuca HbLuca moved this to Backlog in SOS-RS May 20, 2024
@HbLuca HbLuca moved this from Backlog to Disponível pra dev in SOS-RS May 20, 2024
@rhuam rhuam moved this from Disponível para dev to Em revisão (dev) in SOS-RS May 21, 2024
@rhuam
Copy link
Contributor

rhuam commented May 22, 2024

#270

@rhuam
Copy link
Contributor

rhuam commented May 22, 2024

@Tiago-Silva pode verificar os comentários.

Tiago-Silva added a commit to Tiago-Silva/SOS-RS-frontend that referenced this issue May 22, 2024
@rhuam
Copy link
Contributor

rhuam commented May 23, 2024

Falta uma revisão de código, quem pode fazer? @SOS-RS/backenders

@larissapissurno larissapissurno moved this from Em revisão (dev) to PR Aprovado e Mergeado in SOS-RS Jun 4, 2024
@larissapissurno larissapissurno moved this from PR Aprovado e Mergeado to Disponível para QA in SOS-RS Jun 18, 2024
@HbLuca
Copy link

HbLuca commented Jun 19, 2024

@larissapissurno ao realizar o teste deste, apresentou alguns erros (Em Producao) Vou reabri-lo e deixar para DEV verificar.

1 - Ao deixar com filtros, realizar a busca e depois clicar em limpar filtros, a pagina ate limpa, mas segue com o Limpar filtros e a URL como: https://sos-rs.com/?search=
Isto ja foi visto em uma outra issue, depois procuro com calma e a adiciono aqui.
2 - Status do Item do abrigo: Precisa com Urgencia - Status do Abrigo (seleciono os 3) Filtro os resultados e aparecem 3 abrigos.
Deixo apenas o Status do Item do abrigo: Precisa com Urgencia e aparecem 4 abrigos.
Como me aparece 1 a mais sendo que o Status do Item do abrigo es o mesmo?

Comparativo abrigos

Vide Loom:
https://www.loom.com/share/cb046e51ef4e4beb9ba685142ead8cd9?sid=c1d50fb4-2823-4c00-846f-3fe1697982c9

@HbLuca HbLuca reopened this Jun 19, 2024
@github-project-automation github-project-automation bot moved this from Disponível para QA to Backlog in SOS-RS Jun 19, 2024
@HbLuca HbLuca moved this from Backlog to Disponível para dev in SOS-RS Jun 19, 2024
@Tiago-Silva
Copy link
Contributor

Boa noite @HbLuca;
Veja o que eu fiz para corrigir:
Gravação de tela de 22-06-2024 20:26:55.webm

Nos meus testes o que você mencionou na parte 2 não aconteceu, funcionou tudo 100%. Se olhar com calma verá que a mudança de código para isso foi mínima, não tendo impacto no retorno dos dados. Apenas utilizando o que já existia para alterar o titulo.
Exemplo: se o filtro for aplicado mostra um titulo, caso não mostre outro.

@HbLuca
Copy link

HbLuca commented Jun 23, 2024

@Tiago-Silva, bom dia.

Pelo que analisei do video, foi ao contrario...
O 1 foi realizado com sucesso, por limpar os filtros e a url ficar de acordo, alem da pagina.

O 2 nao...
Ao selecionar o Status do item no abrigo: Precisa com urgencia e os 3 Status do Abrigo, aparecem 31 abrigos, ao retirar os 3 Status do Abrigo, aparecem 38 abrigos.
Logo, segue incorreto.

Para que fosse correto, sendo que existem apenas 3 Status do Abrigo, deveriam aparecer os outros 7 em um deles e nao como vazio/0 ou null, referente ao Status do Abrigo.

@Tiago-Silva
Copy link
Contributor

Bom dia @HbLuca;

Eu estava analisando o segundo erro (bug) e aparentemente o back end já retorna o resultado da consulta com error, ou seja, não buscando corretamente ou pelo que mim parece mais certo não está aplicando a mesma lógica para buscar os dados que no frontend está sendo usado para exibir os status dos abrigos.
No frontend é feito apenas a análise do retorno para lapidar as informações para que fique mais legível para o usuário, um exemplo são os status dos abrigos:
Captura de tela de 2024-06-24 10-09-53

Então acredito que para ajustar esse error ao aplicar os filtros deve-se abrir uma tarefa para o backend. Porque nos teste que eu realizei o frontend mostra corretamente o retorno do backend, ou seja, com eu mencionei o backend já retorna os valores controversos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Disponível para dev
Development

Successfully merging a pull request may close this issue.

4 participants