Skip to content

Open training favorites from widget configuration links - #14

Closed
AlexSchmidt1999 wants to merge 2 commits into
codex/training-widget-alignmentfrom
codex/training-widget-configure
Closed

AlexSchmidt1999 wants to merge 2 commits into
codex/training-widget-alignmentfrom
codex/training-widget-configure

Conversation

@AlexSchmidt1999

Copy link
Copy Markdown
Owner

Tapping Configure on an unassigned training favorite previously did nothing because the tile was a disabled Start button. It now opens the existing Training favorites editor directly as a More-tab submenu. Empty training widgets and unassigned Lock Screen favorites open the same editor; configured favorites keep their existing Start intents.

The native widget link uses the existing NOOP URL scheme and NavRouter. The request waits for mandatory launch gates and automatic launch sheets, is consumed once, and resets the More path to a single editor. Opening configuration dismisses ordinary training screens without ending the active session. The editor also remains available under More → Settings. No new storage, dependencies or permissions are introduced.

This draft is stacked on #13 and targets the owner's fork only. Build 440 combines the existing fork test changes with this fix.

Validation:

  • macOS app compiled; all 10 focused TrainingSnapshotTests and ActiveWorkoutIndicatorTests passed, including deferred/one-time configuration routing and unrelated-route preservation.
  • iOS Release app, widget and watch targets compiled successfully.
  • Translation coverage, doc-comment hygiene and source/binary security policy checks passed. The source baseline change reviews only the project build counter; capabilities, dependencies, executables and endpoint sets are unchanged.
  • Build 440 was signed, installed and launched over Wi-Fi on the paired iPhone; the installed version was verified. The configuration URL launch also succeeded.

The visible editor and actual cold/warm widget taps still require a physical iPhone check. Device installation evidence remains local; no upstream PR or release was created.

@AlexSchmidt1999

Copy link
Copy Markdown
Owner Author

Closing this older fork test PR. Its full head (3825d72bfb34) is included in the combined test branch. New test-version work continues in #28. The branch and commits are kept.

The relevant changes are now submitted for upstream review in #2761.

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