armand1m / armand1m/tript-api

Controller de Pesquisa não existe.

Open
#10 0 comments 0 reactions 0 assignees View on GitHub
enhancement question
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Pesquisa não é uma coisa definida somente a veículo. Pesquisa pode ser de veículo, de outras fornecedores, fornecedores proximos, clientes proximos, varias coisas.

A linha 7 do arquivo `./server/routes/search.js` não faz muito sentido, sendo que esta usando um controller de veículos:

``` js
// linha 7
const controller = app.controllers.veiculo;
```

A linha 10 também não faz sentido, sendo que a rota "root" ja busca por veículos, e somente.

``` js
// linha 10
router.get('/', controller.search);
```

O certo seria existir uma rota `search/veiculos` onde chamaria um método de um controller exclusivo (`controller.searchVeiculos()` por exemplo) para pesquisa que ofereceria métodos de pesquisa para os mais diversos tipos de entidades da aplicação.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.