Skip to content

Pairing 6/6: compare daily and workout capabilities - #696

Draft
mikenrafter wants to merge 9 commits into
OpenStrap:mainfrom
mikenrafter:split/669-comparison
Draft

mikenrafter wants to merge 9 commits into
OpenStrap:mainfrom
mikenrafter:split/669-comparison

Conversation

@mikenrafter

@mikenrafter mikenrafter commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Section 6 of the split requested in #669. Draft until preceding sections land. This upstream PR temporarily includes those dependencies; review this section through the link below.

Add the device capability table, daily/workout indicators and the paired-device card. Claims come from the existing compute input rules, with conditional or unverified capabilities labelled and absent inputs left absent.

This optional presentation section is last, as Abdul requested. It can be deferred independently of the identity fix and pairing help. No analytics implementation or version changes are included.

Individual section diff.

Validation: local localization generation and full flutter analyze --no-pub pass. Focused test files pass: slot_comparison_test, ui2_pairing_blocks_test, ui2_device_slot_card_test, ui2_pairing_review_test, ui2_pair_device_settings_test and ui2_tokens_test. Physical iPhone testing remains outstanding.
Upstream review order: 1 ready → 2 draft → 3 draft → 4 draft → 5 draft → 6 draft. All target upstream main; draft diffs include preceding dependencies. Use the individual section link above to review this section alone.

Screenshots

Flutter widget renders using fixture data and the bundled Manrope font. Native iPhone testing remains outstanding.

Daily and workout capability comparison

Comparison sheet pairing actions

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor
Pre-merge checks: Passed 1 · Inconclusive 1

Failed Failed checks: 1 inconclusive

Check name Status Explanation Resolution
Title check Pending This assessment has not completed. The check will be retried on the next pre-merge check run.
Passed Passed checks: 1 passed

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
  • 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.

Share

This branch has not been deployed

No deployments
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.

1 participant