Skip to content

fix(sync): treat optional empty responses as no data - #24

Merged
vicentereig merged 1 commit into
vicentereig:mainfrom
hxu:hxu/sync-optional-endpoints
Jul 27, 2026
Merged

vicentereig merged 1 commit into
vicentereig:mainfrom
hxu:hxu/sync-optional-endpoints

Conversation

@hxu

@hxu hxu commented Jul 22, 2026 •

Copy link
Copy Markdown

Summary

  • Treat Garmin endpoints that legitimately return empty or no-content responses as no data.
  • Use the optional JSON response path for sync calls where an empty response should not fail the task.
  • Avoid sync failures caused by optional endpoints returning empty bodies.

This should build on #21 to actually use the new endpoints and not break the task when we encounter an empty response

Tests

  • cargo test

@vicentereig
vicentereig merged commit ab85ff7 into vicentereig:main Jul 27, 2026
1 check passed
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.

2 participants