Repository navigation
add pickleball to the activity picker - #651
Conversation
a WHOOP import could bring in pickleball sessions, but the picker had no pickleball, so they showed as an unknown activity and couldn't be logged by hand. it sits with the racquet sports, with no calorie rate, like padel. apple health gets its own pickleball type; health connect has none, so it lands as tennis there.
|
@coderabbitai review |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Catalogue and match classification lib/ui2/activity/catalogue.dart, lib/ui2/activity/summary.dart |
The Sports catalogue adds Pickleball as a duration-tracked activity with no MET value. The summary maps it through the match archetype path. |
Health export and Russian label lib/health/health_export.dart, lib/l10n/app_ru.arb |
Health export maps pickleball to PICKLEBALL on iOS and TENNIS on Android. Russian localization adds the pickleball label. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Feature
Possibly related PRs
- OpenStrap/edge#643: Updates activity-name normalization and alias mappings for Apple Health and Health Connect exports.
Suggested labels: Review effort 1/5
Suggested reviewers: droptabl, tahir101981-creator
Merge Risk: 
f19f7
-
Pickleball support is consistent across the activity catalogue, health exports, and Russian localization, with no actionable merge risk identified.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely describes the main change: adding pickleball to the activity picker. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
Sorry @abdulsaheel, you've used your own review budget of 250,000 diff characters for the last 7 days. You can request another review in 5 days and 5 hours by commenting |
|
Reviewer's GuideThe PR introduces Pickleball as a first-class activity in the picker and WHOOP-import catalogue, intentionally leaves its calorie rate unset so estimates use heart rate, categorizes it with racquet sports, and exports it as native Pickleball on iOS or Tennis through Health Connect. Flow diagram for Pickleball activity handlingflowchart LR
WHOOP[WHOOP import] --> Pickleball[Pickleball activity]
Picker[Activity picker] --> Pickleball
Pickleball --> Sports[Sports summary category]
Pickleball --> HR[Heart-rate calorie estimate]
Pickleball --> Export[Health export]
Export -->|iOS| Apple[Apple Health Pickleball]
Export -->|Health Connect| Tennis[Health Connect Tennis]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Failed to generate code suggestions for PR |
|
@coderabbitai review |
✅ Action performed
|
pickleball sessions came in from a WHOOP import but there was no pickleball in the picker, so they showed up as an unknown activity and there was no way to log one by hand.
it sits with the racquet sports. like padel it has no calorie rate, since the compendium the catalogue uses has no pickleball row, so calories come from heart rate. apple health gets its own pickleball workout type; health connect doesn't have one, so it lands as tennis there, same as padel.
Summary by Sourcery
Support pickleball across activity selection, imported workouts, summaries, and health exports.
New Features:
Bug Fixes:
Enhancements:
Tests:
Summary by CodeRabbit