Closed felihenrique closed 7 years ago
@felihenrique, há algo que você queira reportar sobre esse pull request? Ficou algo sem fazer, algo sem testar, alguma grande modificação? Dei uma olhada superficial nas modificações (alterou os construtores dos models também), mas queria que você comentasse algo antes de o @rodrigondec (ou você mesmo) meter o merge.
Resumindo: tô precisando de um feedback mais objetivo à respeito da lista de "doing" aqui: https://github.com/discentes-imd/IntegraTI-API/projects/1
Tudo que está ai está funcionando. Só ta faltando a busca por eventos, e a validação dos modelos de entrada, mas isso é incremental, não tem problema em dar merge.
Alias, está funcionando pelo que testei. Seria bom outras pessoas testarem também.
E os modelos alterei os construtores pelo motivo que tinha te falado. Precisa ser assim porque faço o preenchimento automático do modelo, baseado no json que vem do cliente. Em relação ao "feedback mais objetivo", não entendi.
Controllers para event e event_type.