Feature/corrigir listagem sensores #69
Conversation
filipealvess
added
bug
There was a problem hiding this comment.
Tá no caminho certo, @lopesmayk!! 👏🏻🔥
Além dos comentários abaixo, dá uma olhada nessas coisinhas aqui:
O conteúdo do for pode ser resumido em apenas data.append(sensor), o restante vc pode apagar (essa conversão que tá sendo feita no for já é realizada na função generate_response).
Ah, tem um print() na linha 55 que não deveria existir no repositório, remove ele tbm.
| @@ -39,8 +39,8 @@ def create(): | |||
| except: | |||
| return GlobalController.generate_response(HTTP_BAD_REQUEST_CODE, ERROR_MESSAGE) | |||
|
|
|||
| def list(): | |||
| sensors_list = sensors.find() | |||
There was a problem hiding this comment.
Porque vc alterou o nome dessas variáveis, @lopesmayk?
Pode deixar os nomes que estavam.
Inclusive, a sensors referencia a coleção de sensores do MongoDB (linha 13), mudar ela pra sensors_zones dá erro
app/controllers/sensor_controller.py
Outdated
| def list(user_id): | ||
| sensors_zone_list = sensors_zones.find(user_id) |
There was a problem hiding this comment.
Como essa função tá acessando um serviço externo (MongoDB), a gente tem que levar em consideração que pode acontecer um erro inesperado.
Então, pra se preparar pra esse cenário, envolve a lógica da função em um try/except, semelhante à função create() da linha 18.
app/controllers/sensor_controller.py
Outdated
| def list(user_id): | ||
| sensors_zone_list = sensors_zones.find(user_id) |
There was a problem hiding this comment.
A chamada do find tá errada. Vc tem que passar um dicionário ({}) com a propriedade e o valor que vc quer consultar na coleção.
Como vc tá buscando pelo ID do usuário, tem que ser nesse estilo:
sensors.find({ 'user_id': user_id })
|
@lopesmayk, quando vc for fazer as correções, siga esse passo a passo: Aliás, antes de tudo, verifique se a branch ativa é a mesma do PR: git branch
# vai aparecer algo assim:
* feature/corrigir-listagem-sensores
main
# a branch que tem o asterisco é a branch ativaSe a branch for diferente, altere pra ela: git checkout feature/corrigir-listagem-sensoresAgora sim pode seguir o passo a passo tranquilamente:
@JoaoViniciusLima, quando tu for fazer as correções nos teus PRs, siga essa linha tbm 😉 |
…o feature/corrigir-listagem-sensores
lopesmayk
force-pushed
the
feature/corrigir-listagem-sensores
branch
from
7d7d82a
to
4e111ae
Compare
corrigi a listagem dos sensores