search

yihong0618 / running_page

Make your own running home page
https://running-page.vercel.app
MIT License
3.55k stars 1.02k forks source link

feat: Sync garmin cn to global #583

Closed NaturezzZ closed 9 months ago

NaturezzZ commented 9 months ago

This script downloads all .fit files from Garmin CN and uploads them to Garmin Global. It is worthwhile because Garmin Global can automatically sync with Strava with more types of sports, more accurate device info and running statistics.

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
running-page ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 5, 2024 5:20am
yihong0618 commented 9 months ago

Thanks, this is a huge feature, I want to do it a long time ago, thanks very much it will help many people

Will take a look later

yihong0618 commented 9 months ago

can you add some desc to README and README-CN/?

NaturezzZ commented 9 months ago

definitely ok

NaturezzZ commented 9 months ago

@yihong0618 @ben-29 I rewrote this module. Now synchronizing Garmin CN to global is in two steps. Step 1: Download new activities from Garmin CN in FIT format (if no FIT, use gpx instead). Upload to Garmin global. Step 2: Download new tracks in gpx and generate running page with them. Note that I did not generate running page with FIT because [fit-tool crashes when parsing some FIT files].(https://github.com/yihong0618/running_page/issues/584)

Syned activity list is recorded in synced_activity.json.

NaturezzZ commented 9 months ago

I think we should consider about whether to fix issue #584 first. This PR works without the need to fix issue #584, but fixing it makes this PR better.

yihong0618 commented 9 months ago

I think we should consider about whether to fix issue #584 first. This PR works without the need to fix issue #584, but fixing it makes this PR better.

OK, will dig it later

ben-29 commented 9 months ago

@NaturezzZ I'm sorry for the delay. Let's continue with this PR. Please make the necessary adjustments for step 2. Thank you.

NaturezzZ commented 9 months ago

@ben-29 I think it is ok now. Could you help me verify if the issue has been resolved?

yihong0618 commented 9 months ago

@NaturezzZ thanks let us use squash merge next time ~

NaturezzZ commented 9 months ago

Sorry~ I forgot it. Next time will be better!

Allianzcortex commented 9 months ago

I think this repo settings should be only allowed using squash & merge, very sure it's doable. My company uses terraform to manage it, and it can also be configurable via github settings.

yihong0618 commented 9 months ago

I think this repo settings should be only allowed using squash & merge, very sure it's doable. My company uses terraform to manage it, and it can also be configurable via github settings.

Thanks will do this.