tchx84 / linka

An open source service to record air quality data from community sensors
GNU Affero General Public License v3.0
21 stars 12 forks source link

WIP: service: Add stats empoint #61

Closed sofiab97 closed 3 years ago

sofiab97 commented 3 years ago

date es la fecha que calcule con la hora actual del sistema menos los 30 minutos, para que muestre los datos en ese periodo. No se si ahí es el lugar donde tiene que estar pero como me dijiste el ejemplo de que cargamos los datos desde que naciste, start y end son los que deberían cambiar?

El lun, 26 abr 2021 a las 7:41, Martin Abente Lahaye (< @.***>) escribió:

@.**** requested changes on this pull request.

In tests/service.py https://github.com/tchx84/linka/pull/61#discussion_r620216717:

@@ -135,7 +135,7 @@ def test_invalid_api_key_access():

def test_query(): query = {

  • "start": "1984-04-24T00:00:00",
  • "start": "date",

Que exactamente queres lograr con esto?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#pullrequestreview-644570441, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAQXYVGHZ555APYJVOTTKVGORANCNFSM42ANOURQ .

tchx84 commented 3 years ago

date es la fecha que calcule con la hora actual del sistema menos los 30 minutos, para que muestre los datos en ese periodo.

Los cambios que te pedi son para el front-end de aire libre, no este backend. Tenes esos cambios en tu fork del front-end?

sofiab97 commented 3 years ago

Sii, hice los cambios en el fetch.js Hice el commit ahora! como puedes ver los cambios? hacer git push mine stats no me deja.

El lun, 26 abr 2021 a las 9:37, Martin Abente Lahaye (< @.***>) escribió:

date es la fecha que calcule con la hora actual del sistema menos los 30 minutos, para que muestre los datos en ese periodo.

Los cambios que te pedi son para el front-end de aire libre, no este backend. Tenes esos cambios en tu fork del front-end?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#issuecomment-826842189, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAVVAEILWWULOURIJHDTKVUCVANCNFSM42ANOURQ .

tchx84 commented 3 years ago

Sii, hice los cambios en el fetch.js Hice el commit ahora! como puedes ver los cambios? hacer git push mine stats no me deja.

Copia aca el mensaje que te tira, porque "no me deja" no me dice mucho :)

sofiab97 commented 3 years ago

error: failed to push some refs to 'mine'

El lun, 26 abr 2021 a las 10:21, Martin Abente Lahaye (< @.***>) escribió:

Sii, hice los cambios en el fetch.js Hice el commit ahora! como puedes ver los cambios? hacer git push mine stats no me deja.

Copia aca el mensaje que te tira, porque "no me deja" no me dice mucho :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#issuecomment-826875181, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAROO6MLA5POJQLPDCDTKVZHHANCNFSM42ANOURQ .

tchx84 commented 3 years ago

error: failed to push some refs to 'mine'

puede ser que te falte crear el branch?

$ git checkout -b stats
$ git push mine stats

De todas formas hace git status y copia aca, asi vemos.

sofiab97 commented 3 years ago

Al hacer $ git satus

On branch statsnothing to commit, working tree clean

El lun, 26 abr 2021 a las 10:29, Martin Abente Lahaye (< @.***>) escribió:

error: failed to push some refs to 'mine'

puede ser que te falte crear el branch?

$ git checkout -b stats $ git push mine stats

De todas formas hace git status y copia aca, asi vemos.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#issuecomment-826881220, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAUDQOYXMEHS6PA5YHLTKV2FTANCNFSM42ANOURQ .

tchx84 commented 3 years ago

Al hacer $ git satus On branch statsnothing to commit, working tree clean

Ok, corre los siguientes comandos y copia el output aca:

$ pwd
$ git remote
$ git status
$ git log -n 1
sofiab97 commented 3 years ago

El lun, 26 abr 2021 a las 10:39, Martin Abente Lahaye (< @.***>) escribió:

Al hacer $ git satus On branch statsnothing to commit, working tree clean

Ok, corre los siguientes comandos y copia el output aca:

$ pwd $ git remote $ git status $ git log -n 1

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#issuecomment-826889180, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSASWKFEXB3DI7VN4ERDTKV3LDANCNFSM42ANOURQ .

tchx84 commented 3 years ago

El lun, 26 abr 2021 a las 10:39, Martin Abente Lahaye (< @.**>) escribió: Al hacer $ git satus On branch statsnothing to commit, working tree clean* Ok, corre los siguientes comandos y copia el output aca: $ pwd $ git remote $ git status $ git log -n 1 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSASWKFEXB3DI7VN4ERDTKV3LDANCNFSM42ANOURQ .

ehh, no pegaste nada ahi :confused:

sofiab97 commented 3 years ago

No entiendo..

*commit 0ec155e8cdaabbb1cbf27195787a0fd4c189a2b9 (HEAD -> stats, main)Author: Sofia Benitez @. @.>>Date: Mon Apr 26 10:00:37 2021 -0400 modified: js/fetch.js*

Esto salió al final de todos los comandos.

El lun, 26 abr 2021 a las 10:45, Martin Abente Lahaye (< @.***>) escribió:

El lun, 26 abr 2021 a las 10:39, Martin Abente Lahaye (< @.**>) escribió: … <#m7362334855268968182> Al hacer $ git satus On branch statsnothing to commit, working tree clean* Ok, corre los siguientes comandos y copia el output aca: $ pwd $ git remote $ git status $ git log -n 1 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61 (comment) https://github.com/tchx84/linka/pull/61#issuecomment-826889180>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSASWKFEXB3DI7VN4ERDTKV3LDANCNFSM42ANOURQ .

ehh, no pegaste nada ahi 😕

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tchx84/linka/pull/61#issuecomment-826893466, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAUIQIOJQQXXXODWUFTTKV4ADANCNFSM42ANOURQ .

tchx84 commented 3 years ago

No entiendo.. *commit 0ec155e8cdaabbb1cbf27195787a0fd4c189a2b9 (HEAD -> stats, main)Author: Sofia Benitez @. @.>>Date: Mon Apr 26 10:00:37 2021 -0400 modified: js/fetch.js* Esto salió al final de todos los comandos. El lun, 26 abr 2021 a las 10:45, Martin Abente Lahaye (< @.>) escribió: El lun, 26 abr 2021 a las 10:39, Martin Abente Lahaye (< @.>) escribió: … <#m7362334855268968182> Al hacer $ git satus On branch statsnothing to commit, working tree clean Ok, corre los siguientes comandos y copia el output aca: $ pwd $ git remote $ git status $ git log -n 1 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61 (comment) <#61 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSASWKFEXB3DI7VN4ERDTKV3LDANCNFSM42ANOURQ . ehh, no pegaste nada ahi — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATEDSAUIQIOJQQXXXODWUFTTKV4ADANCNFSM42ANOURQ .

No se entiende nada, entra a https://github.com/tchx84/linka/pull/61 y responde desde ahi please comando por comando.

tchx84 commented 3 years ago

@sofiab97 reminder: you still haven't push the changes