-
Notifications
You must be signed in to change notification settings - Fork 80
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
Add activity typeId filtering #72
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice PR. Maybe I would consider adding tests?
I also leave some code suggestions.
Co-authored-by: Bart Skowron <[email protected]>
@bxsx Appreciate the feedback, will look into adding a test when I have some time. |
Co-authored-by: Bart Skowron <[email protected]>
See also comment <#72 (comment)>
@joetimmerman I had to slightly adapt your code while rebasing, it's now part of the next version, see #104 |
Cheers, thanks for keeping this repo going. |
Adds the ability to filter based on typeIds as requested in #71. The new --type_filter argument allows you to pass a list of typeIds and only pull activities matching those Ids.
For example:
$ python gcexport.py -tf 3
Results in:
Filtering out due to typeID, 1 not in [3] : Garmin Connect activity (1/1) [8791243351