-
Notifications
You must be signed in to change notification settings - Fork 293
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
Update - removendo limite de supplies na listagem de abrigos #136
Update - removendo limite de supplies na listagem de abrigos #136
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Opa @vinny-silveira, acabei de atualizar com a develop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bom dia @xlucaix, você está rodando o backend local? Pois ainda segue sem funcionar:
Atualizei minha branch local com as suas atualizações e mesmo assim o erro persiste:
Antes de submeter a PR ou uma nova revisão, peço que minimamente rode o projeto localmente e garanta o funcionamento básico do projeto.
@vinny-silveira aqui tinha rodado localmente tranquilo, o que foi estranho kkkk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Certo, o problema de execução foi resolvido, porém, as tags não retornam corretamente do BFF, deixando o front dessa forma:
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:
Ou sempre teste em abas anônimas.
Talvez por cache comigo exibia, alterei o sche |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR apto ao merge
@SOS-RS/backenders , preciso de mais um review aqui. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
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: SOS-RS/frontend#249