Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Activity recording redesign - #57

Merged
saksham2001 merged 8 commits into
mainfrom
activity-recording-redesign
Jul 8, 2026
Merged

saksham2001 merged 8 commits into
mainfrom
activity-recording-redesign

Conversation

@saksham2001

@saksham2001 saksham2001 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

Activity Recording was broken. This PR tries to fix some of the issues and adds some features:

  • Fixed GPS data recording
  • Fixed HR data recording, handling connection dropouts, and background syncs
  • Allow activity modification (after competition) to change activity time.
  • Pause while activity is going on
  • Performance optimization to laggy UI
  • Bugs in Live Activity fixed - HR and SpO2 values not visible and live activity never ends
  • Added option in UI to manually log activities.

Related issues

Closes #33

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 📟 New / improved wearable support (BLE driver layer)
  • 🤖 Coach / LLM change (tools, prompts, orchestration)
  • 🎨 UI / DesignSystem change
  • 🧹 Refactor / chore (no behavior change)
  • 📝 Docs only
  • ⚠️ Breaking change (existing data, settings, or APIs change)

How was this tested?

Tested by performing multiple activities.

  • Added / updated unit tests (PulseLoopTests)
  • Ran the test suite locally (⌘U in Xcode)
  • Tested on a physical device with a real ring — model:
  • Tested with demo data (-seedDemo YES, no hardware)
  • N/A (docs / non-code change)

Privacy & data

  • This change does not send health data off-device without explicit user action.
  • No secrets, API keys, or personal data are committed.
  • N/A

Checklist

  • My code follows the project's style (SwiftLint passes).
  • I ran the tests and they pass.
  • I updated docs / README where relevant.
  • I read the Contributing guide.

@saksham2001
saksham2001 merged commit 8182d8d into main Jul 8, 2026
2 checks 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.

[Bug]: Activity recoding is broken

1 participant