Updates rwgps colors too

Author: ckdake

TODO.md

@@ -5,9 +5,7 @@ Tracked here for visibility. PRs and issues welcome.
 ## Overall capabilitites
 
 next:
-- [ ] update rwgps logo assetts https://ridewithgps.com/api/v1/doc/reference/logos_and_assets
 - [ ] for web app, require subscription to be active for providers to sync
-- [ ] user docs -> tracekit will not modify unless you click individual button, FAQ, how to setup
 - [ ] discord?
 - [ ] file provider should have an upload button that uploads a new file. dont overwrite existing files! (check filename on clientside and server side). if its a zip, extrct safely
 
@@ -21,8 +19,7 @@ later
 - [ ] outbound email sending for email verificaiton on login, and password reset
 - [ ] add new source data files to file data when found in other places
 - [ ] i think we need a websocket or something for realtime updates
-- [ ] download file from ridewithgps, strava if file is missing (like garmin)
-- [ ] add inbound webhooks from ridewithgps on trips
+- [ ] add inbound webhooks from ridewithgps on trips -> verify
 
 ## Providers & Sync
 

app/static/style.css

@@ -791,7 +791,7 @@ select#timezone:focus {
 /* Per-provider synced colors */
 .provider-status.provider-strava.synced      { background: #FC4C02; }
 .provider-status.provider-garmin.synced      { background: #007CC3; }
-.provider-status.provider-ridewithgps.synced { background: #52AF4A; }
+.provider-status.provider-ridewithgps.synced { background: #F96400; }
 .provider-status.provider-spreadsheet.synced { background: #5a6b7c; }
 .provider-status.provider-file.synced        { background: #7a7f84; }
 
@@ -1087,15 +1087,15 @@ select#timezone:focus {
     font-size: 1rem;
     padding: 11px 22px;
     border-radius: 4px;
-    background: #52AF4A;
+    background: #F96400;
     color: #ffffff;
     border: none;
     cursor: pointer;
     font-weight: 700;
     letter-spacing: .015em;
     transition: background .15s;
 }
-.ridewithgps-auth-btn:hover { background: #429b3b; }
+.ridewithgps-auth-btn:hover { background: #D95500; }
 
 /* Personal-credentials toggle — displayed subtly below the auth button */
 .personal-creds-toggle-row {
@@ -1191,13 +1191,13 @@ select#timezone:focus {
     color: #005f99 !important;
     text-decoration: underline;
 }
-/* RideWithGPS brand green: #52AF4A */
+/* RideWithGPS brand orange: #F96400 */
 .attr-ridewithgps {
-    color: #52AF4A !important;
+    color: #F96400 !important;
     font-weight: 600;
 }
 .attr-ridewithgps:hover {
-    color: #3d8836 !important;
+    color: #D95500 !important;
     text-decoration: underline;
 }
 
@@ -1225,10 +1225,10 @@ select#timezone:focus {
 }
 .provider-attr.attr-garmin:hover { background: rgba(0,124,195,.08); }
 .provider-attr.attr-ridewithgps {
-    color: #52AF4A;
-    border: 1px solid rgba(82,175,74,.35);
+    color: #F96400;
+    border: 1px solid rgba(249,100,0,.35);
 }
-.provider-attr.attr-ridewithgps:hover { background: rgba(82,175,74,.08); }
+.provider-attr.attr-ridewithgps:hover { background: rgba(249,100,0,.08); }
 
 /* Attribution links inside synced (green) provider cards — use white */
 .provider-status.synced .provider-attr {

app/templates/month.html

@@ -81,6 +81,9 @@ <h2 class="section-title">Pending Changes</h2>
     case 'Download Source File':
       return `Download source file from Garmin for activity ${c.activity_id} `
            + `(<em>${c.new_value || ''}</em>) to file provider`;
+    case 'Download Source File from RideWithGPS':
+      return `Download source file from RideWithGPS for activity ${c.activity_id} `
+           + `(<em>${c.new_value || ''}</em>) to file provider`;
     default:
       return JSON.stringify(c);
   }

pyproject.toml

@@ -17,7 +17,7 @@ dependencies = [
     "dateparser==1.3.0",
     "pytz==2025.2",
     "requests>=2.31.0,<3.0.0",
-    "pyrwgps==0.2.0",
+    "pyrwgps==0.2.1",
     "python-dotenv==1.2.1",
     "stravalib==2.4",
     "flask>=3.0.0,<4.0.0",

tracekit/providers/ridewithgps/ridewithgps_provider.py

@@ -314,6 +314,42 @@ def sync_single_activity(self, trip_id: str) -> RideWithGPSActivity | None:
             print(f"RideWithGPS webhook: created local trip {trip_id}")
             return local
 
+    def download_activity_file(self, rwgps_id: str, dest_dir: str) -> str:
+        """Download an activity source file from RideWithGPS.
+
+        Tries formats in preference order: TCX, GPX, KML.  Saves the file
+        under *dest_dir* with a stable name derived from *rwgps_id*.
+
+        Returns the full path of the saved file.
+
+        Raises:
+            FileExistsError: if the destination file already exists (never overwrites).
+            RuntimeError: if the activity cannot be downloaded in any format.
+        """
+        import os
+
+        os.makedirs(dest_dir, exist_ok=True)
+
+        for fmt in ("tcx", "gpx", "kml"):
+            try:
+                content = self.client.download_trip_file(rwgps_id, fmt)
+                if content:
+                    dest_path = os.path.join(dest_dir, f"ridewithgps_{rwgps_id}.{fmt}")
+                    if os.path.exists(dest_path):
+                        raise FileExistsError(f"File already exists: {dest_path}")
+                    with open(dest_path, "wb") as f:
+                        f.write(content)
+                    print(f"Downloaded {fmt.upper()} for RideWithGPS trip {rwgps_id}: {dest_path}")
+                    return dest_path
+            except FileExistsError:
+                raise
+            except Exception as e:
+                print(f"Could not download {fmt.upper()} for RideWithGPS trip {rwgps_id}: {e}")
+
+        raise RuntimeError(
+            f"Could not download RideWithGPS trip {rwgps_id} in any supported format (tried TCX, GPX, KML)"
+        )
+
     def reset_activities(self, date_filter: str | None = None) -> int:
         """Reset (delete) RideWithGPS activities from local database."""
         from tracekit.providers.ridewithgps.ridewithgps_activity import (

tracekit/sync.py

@@ -33,6 +33,7 @@ class ChangeType(Enum):
     ADD_ACTIVITY = "Add Activity"
     LINK_ACTIVITY = "Link Activity"
     DOWNLOAD_FROM_GARMIN = "Download Source File"
+    DOWNLOAD_FROM_RIDEWITHGPS = "Download Source File from RideWithGPS"
 
 
 class ActivityChange(NamedTuple):
@@ -74,6 +75,11 @@ def __str__(self) -> str:
                 f"Download source file from Garmin for activity {self.activity_id} "
                 f"('{self.new_value}') to file provider"
             )
+        elif self.change_type == ChangeType.DOWNLOAD_FROM_RIDEWITHGPS:
+            return (
+                f"Download source file from RideWithGPS for activity {self.activity_id} "
+                f"('{self.new_value}') to file provider"
+            )
         return "Unknown change"
 
     def to_dict(self) -> dict:
@@ -407,6 +413,22 @@ def compute_month_changes(
                     )
                 )
 
+    # Same for RideWithGPS.
+    if "ridewithgps" in provider_priorities and "file" in provider_priorities:
+        for _key, group in grouped.items():
+            by_provider = {a["provider"]: a for a in group}
+            if "ridewithgps" in by_provider and "file" not in by_provider:
+                rwgps_act = by_provider["ridewithgps"]
+                all_changes.append(
+                    ActivityChange(
+                        change_type=ChangeType.DOWNLOAD_FROM_RIDEWITHGPS,
+                        provider="file",
+                        activity_id=str(rwgps_act["id"]),
+                        new_value=rwgps_act["name"],
+                        source_provider="ridewithgps",
+                    )
+                )
+
     return dict(grouped), all_changes
 
 
@@ -693,6 +715,27 @@ def apply_change(change: ActivityChange, tracekit: Tracekit, grouped: dict | Non
                 return True, f"Downloaded and ingested {result.get('file')}"
             return False, f"Download succeeded but ingestion failed: {result.get('reason', 'unknown')}"
 
+        elif change_type == ChangeType.DOWNLOAD_FROM_RIDEWITHGPS:
+            rwgps_prov = tracekit.get_provider("ridewithgps")
+            file_prov = tracekit.get_provider("file")
+            if not rwgps_prov:
+                return False, "RideWithGPS provider not available"
+            if not file_prov:
+                return False, "File provider not available"
+
+            rwgps_id = change.activity_id
+            dest_dir = file_prov.data_folder
+
+            try:
+                file_path = rwgps_prov.download_activity_file(rwgps_id, dest_dir)
+            except FileExistsError as exc:
+                return False, f"File already exists — will not overwrite: {exc}"
+
+            result = file_prov.process_single_file(file_path)
+            if result.get("status") in ("ok", "skipped"):
+                return True, f"Downloaded and ingested {result.get('file')}"
+            return False, f"Download succeeded but ingestion failed: {result.get('reason', 'unknown')}"
+
         else:
             return False, f"Unsupported change type: {change_type}"