search

caronae / caronae-backend

[Projeto descontinuado] Backend do Caronaê, app open-source de caronas usado por mais de 15 mil alunos da UFRJ
https://caronae.org
GNU General Public License v3.0
38 stars 16 forks source link

Listar apenas caronas da instituição do usuário #6

Closed macecchi closed 6 years ago

macecchi commented 7 years ago

O usuário do Caronaê deve poder visualizar apenas as caronas oferecidas na sua própria instituição.

Exemplo: se sou aluno da UFRJ, não posso visualizar caronas ofertadas por alunos da UNIRIO.

Para isso, a listagem de caronas que vem da API deve levar em consideração a instituição do usuário que fez a requisição e retornar apenas as caronas da instituição a que pertence.

Detalhes

Cada usuário já é associado a uma instituição e cada carona também. É necessário adaptar a rota de listar/pesquisar caronas (RideController::index) para filtrar apenas as caronas que pertencem a motoristas da mesma instituição do usuário atual.

Extras

A rota de requestJoin (por onde pode solicitar participar numa carona) também deve impedir que um usuário de uma instituição solicite participar de uma carona de outra instituição.

macecchi commented 7 years ago

Listar caronas apenas da instituição do usuário

danielsouzaaf commented 7 years ago

oi @macecchi, tô dando uma olhadinha nessa issue!

macecchi commented 7 years ago

Oi @danielsouzaaf! Muita massa! Se quiser podemos trocar uma ideia melhor pra tirar dúvidas, ou comenta aqui mesmo qualquer dúvida que tiver! \o/

macecchi commented 7 years ago

@danielsouzaaf e aí, tá conseguindo se entender com o código? me fala aí se precisar de alguma coisa! ah, e caso ache que não vá conseguir fazer, nos avisa que passamos pra outra pessoa :)

danielsouzaaf commented 7 years ago

fala @macecchi, acabou que quando fui setar o ambiente de dev deu um erro nos seeders, fiquei de ver um workaround para poder testar e me familiarizar com o sistema mas não deu em nada. :/ se quiser deixar para outra pessoa, fica a seu critério, mas acho que vou dar mais uma olhadinha sim

macecchi commented 7 years ago

@danielsouzaaf putz :/ Pior que os seeders tão quebrados mesmo. Eu até criei o #7 pra listar isso. Vou ver aqui se consigo pegar o #7 então pra resolver isso e aí te desbloqueio. Te aviso quando tiver um progresso!

macecchi commented 6 years ago

Implementado no PR #27. Valeu @rpaggi !