SOS-RS / backend

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

Update - removendo limite de supplies na listagem de abrigos #136

Closed xlucaix closed 3 months ago

xlucaix commented 3 months ago

Melhorar a descoberta de que existem mais itens depois que o limite de 10 é excedido no card

Conforme solicitado na issue #257 foi removido o limite no retorno dos supplies de abrigo. PR do Frontend: https://github.com/SOS-RS/frontend/pull/249

xlucaix commented 3 months ago

O nome do método getQuery foi alterado mas a chamada dele no service não foi ajustada, está dando erro logo ao iniciar o projeto:

image

Opa @vinny-silveira, acabei de atualizar com a develop

xlucaix commented 3 months ago

@vinny-silveira aqui tinha rodado localmente tranquilo, o que foi estranho kkkk Depois de comparar o arquivo shelterSearch notei que a classe foi alterada, acredito ter sido erro aqui de alguma extensão local, vou revisar depois qual pode ter feito isso. Subi um novo commit removendo as alterações da classe e já rodando o projeto

xlucaix commented 3 months ago

Certo, o problema de execução foi resolvido, porém, as tags não retornam corretamente do BFF, deixando o front dessa forma:

image

Você removeu o defaultTagsData, e pelo objeto tags da linha 176 ficar vazio, nada é tagueado. Para manter o funcionamento, sugiro que volte o objeto defaultTagsData e faça o spread dele como era feito antes:

  const tags: ShelterTagInfo = {
    ...defaultTagsData,
    ...(tagProps?.tags ?? {}),
  };

Novamente, peço que teste o backend integrado com o front sempre, e para evitar caches, sempre faça um clear no Chrome pelo Developer Tools:

image

Ou sempre teste em abas anônimas.

Talvez por cache comigo exibia, alterei o sche ma da tag para boolean, já que o objetivo náo é lidar mais limitar quantidades, acho que agora ta ok, testei várias vezes removendo cache.

Screenshot 2024-05-22 at 18 19 47

vinny-silveira commented 3 months ago

@SOS-RS/backenders , preciso de mais um review aqui.