Skip to content

fix(backend): update last_synced_at when data arrives via webhook#590

Draft
bartmichalak wants to merge 1 commit intomainfrom
fix/update-last-synced-at-when-data-are-updated-via-garmin-webhooks
Draft

fix(backend): update last_synced_at when data arrives via webhook#590
bartmichalak wants to merge 1 commit intomainfrom
fix/update-last-synced-at-when-data-are-updated-via-garmin-webhooks

Conversation

@bartmichalak
Copy link
Contributor

Description

  • Garmin webhook endpoints (push/ping) processed and saved incoming data correctly but never updated last_synced_at on the user connection
  • Added tracking of user IDs that successfully received data during webhook processing
  • After commit, last_synced_at is now updated for all affected connections

Checklist

General

  • My code follows the project's code style
  • I have performed a self-review of my code
  • I have added tests that prove my fix/feature works (if applicable)
  • New and existing tests pass locally

Backend Changes

You have to be in backend directory to make it work:

  • uv run pre-commit run --all-files passes

Frontend Changes

  • pnpm run lint passes
  • pnpm run format:check passes
  • pnpm run build succeeds

Testing Instructions

Steps to test:
1.
2.
3.

Expected behavior:

Screenshots

Additional Notes

@coderabbitai
Copy link

coderabbitai bot commented Mar 10, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 23048421-1de9-4989-bf2c-e8b081ffd208

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/update-last-synced-at-when-data-are-updated-via-garmin-webhooks

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@railway-app
Copy link

railway-app bot commented Mar 10, 2026

🚅 Deployed to the open-wearables-pr-590 environment in Open Wearables

Service Status Web Updated (UTC)
frontend ✅ Success (View Logs) Web Mar 10, 2026 at 11:43 pm
celery-flower ✅ Success (View Logs) Web Mar 10, 2026 at 11:43 pm
celery-worker ✅ Success (View Logs) Mar 10, 2026 at 11:43 pm
celery-beat ✅ Success (View Logs) Mar 10, 2026 at 11:42 pm
backend ✅ Success (View Logs) Web Mar 10, 2026 at 11:42 pm

@bartmichalak bartmichalak linked an issue Mar 10, 2026 that may be closed by this pull request
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

Successfully merging this pull request may close these issues.

Garmin webhooks do not update last_synced_at on user connection

1 participant