backend icon indicating copy to clipboard operation
backend copied to clipboard

Update - removendo limite de supplies na listagem de abrigos

Open xlucaix opened this issue 1 year ago • 4 comments

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 avatar May 18 '24 20:05 xlucaix

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 avatar May 21 '24 21:05 xlucaix

@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 avatar May 22 '24 17:05 xlucaix

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

xlucaix avatar May 22 '24 21:05 xlucaix

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

vinny-silveira avatar May 23 '24 12:05 vinny-silveira