-
Notifications
You must be signed in to change notification settings - Fork 61
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
exception with body_fat data type #1
Comments
WHen accessing the link in a browser, I get the following error:
With the steps followed, I was under the impression that the auth aspect was complete |
@rickysarraf , It seems like the exception is caused by body_fat sync. Could you please try against the latest commit? If you still face the issue, try setting Please post me a log if the problem persists. I just tried a full sync from my end without any errors. |
@praveendath92 That was my first suspect.
To cut system libs out, I also did pip installations. BTW, you may want to explicitly define "/usr/bin/env python3" because many distros still default to python2. But with latest pip3 install libs too did not help.
|
THere's indentation issues with your commit.
But that did not help either.
|
I think dependencies are not an issue. To sync data with Google Fit, the app should first create data source for each type of data. The log says there's a bad request when creating data source for body_fat. Strange thing is that it doesn't happen to me! |
Do you know why the URL, when opened, says "Login Required" ? I don't think the issue may be with your code (given that you confirmed that it is working for you). But rather, with the workflow you've documented. Are the 2 json files enough for the auth or anything else is also required ? |
Just the json files are enough. "Login Required" message is normal for Google API endpoints when opened in browser. The authentication is done through |
I got it. Let me send you a quick fix. |
In this line - https://github.com/praveendath92/fitbit-googlefit/blob/master/app.py#L57 Could you try removing |
The app creates data source for all types whether or not sync for that type is enabled. |
I could reproduce the error with a different account. I will work on fix. Until then, this fix solves the error. |
That worked. But I got a different error now.
|
Would you mind if we discuss this as a new issue? |
Hi, Thanks for creating this bridge.
When trying to run app.py, I've run into the following exception. I did setup the fitbit/google authentication successfully, as you've documented.
The text was updated successfully, but these errors were encountered: