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

Impedir que caronas duplicadas sejam criadas #10

Closed macecchi closed 6 years ago

macecchi commented 7 years ago

A API deve impedir que o usuário ofereça mais de uma carona na mesma data.

Foi implementada uma rota (em RideController::validateDuplicate) para a verificação prévia se uma carona é duplicada ou muito parecida com outra já criada.

No entanto, a API não está verificando esse estado no momento da criação, o que torna possível para os clientes ainda criarem caronas duplicadas e não seguir a verificação prévia.

A API deveria retornar um estado de erro para o caso em que a nova carona é considerada duplicada (duplicated).

No caso de a carona ser considerada apenas possible_duplicate, o servidor deverá permitir que seja criada mesmo assim.

macecchi commented 7 years ago

Impedir que caronas duplicadas sejam criadas

CauanCabral commented 6 years ago

@macecchi estou dando uma olhada no projeto para tentar ajudar nessa issue. O requisito continua o mesmo?

CauanCabral commented 6 years ago

Acho que o PR #23 já resolveu bem o problema

macecchi commented 6 years ago

Mergeado o #23 mesmo. De qualquer maneira, obrigado @CauanCabral!

munizeverton commented 6 years ago

@CauanCabral cara, só vi que você estava trabalhando nisso depois de abrir o PR :disappointed:

Sorry

CauanCabral commented 6 years ago

hehehe, tranquilo @munizeverton , o importante é ajudar o projeto ;)