Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Date problem building daily activity summary options #12

Open
BarrieP opened this issue Apr 7, 2016 · 2 comments
Open

Date problem building daily activity summary options #12

BarrieP opened this issue Apr 7, 2016 · 2 comments

Comments

@BarrieP
Copy link

BarrieP commented Apr 7, 2016

Currently sets the date as :
date: moment().format('YYYY-MM-DD'),

If the user isn't in the same timezone as the server, this will result in the data getting reset to zero at the wrong time ( from the users perspective ).

It seems to work fine just setting :
date: 'today'

Although I've not tested the rollover yet, this would be Fitbit's problem if it goes wrong...

@ghost
Copy link

ghost commented Apr 11, 2016

PRs are welcome

@BarrieP
Copy link
Author

BarrieP commented Apr 11, 2016

I hope to have time in the next few weeks ( need to align with the next-release branch rather than master I guess... I might have some more APIs I can implement for it too, but not sure what is in that branch ).

In the meantime I thought I'd at least share my debugging effort.

On that note, using 'today' seems inconsistent... it seems like it might work AOK for the daily summary object, but doesn't always return the goals object.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant