-
Notifications
You must be signed in to change notification settings - Fork 321
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
feat: add home supply rank ISSUE-64 (Totalizador de pedidos de doação) #67
base: develop
Are you sure you want to change the base?
Conversation
develop into master
*Fix: mobile UI has broken for mobile devices * fix: remove chip as fixed as no wrap property * chore: add shelter card clickable * chore: remove Fragment component * build: add set sm as max mobile dimension sm: 425px * style: add responsive layout
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.
Seria bacana criar um espaçamento entre o titulo e as tags dos items, isso iria facilitar a leitura do usuário.
Um melhor local para implementar esta seção na minha opinião seria abaixo dos filtos.
criando uma ordem entre abrigos e filtro de abrigos. e depois a seção de items.
Sim o espaçamento seria interessante, você tem interesse em adicionar @okayosilva.? Acredito que mudar o local para abaixo dos filtros pode trazer confusão e fazer o usuário achar que essas dados tem vinculo com o filtro ou a busca. |
Co-authored-by: Lucas Inocente <olucassanchez@gmail.com>
o que falta para aprovarmos pessoal? A gente precisa da info o quanto antes, ali no topo, qualquer coisa diferente disso é "perfumaria" |
Acredito que não se trata apenas de 'perfumaria', mas sim de adotar um pensamento assertivo para comunicar as informações de forma mais eficaz. Para aprovação, é necessário que alguém da equipe o faça. |
Co-authored-by: Luciano Lima <beavess@gmail.com>
Pessoal, acho todos os comentários úteis, mas dado a situação (não preciso falar aqui) vocês tem que ser mais abertos a alterações que não tem impacto ao usuário final. Esse sistema foi feito para resolver um problema urgente e não podemos estar adicionando burocracia. Acelerar o processo é a chave e isso vale para todos os pull requests. |
Lembrando que para subir este MR precisamos antes resolver a issue do backend: SOS-RS/backend#38 |
Foi fechada já! 🥰 |
rota no back end já está pronta? |
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.
[DUVIDA] - Nós não pegamos essas infos de acordo com cada abrigo?
SOS-RS/backend#29 talvez esse link te ajude |
Para liberar esse pull request será necessário implementar no backend uma uma nova rota
GET https://api.sos-rs.com/supply/top10 com a resposta no seguinte formato:
{"data":[{"name": "Agua 5L", "amount": 10}, {"name": "Cobertor", "amount": 5}]}
ISSUE #64