Skip to content

Phase 2: Remove page-level service location - #715

Merged
StuartFerguson merged 1 commit into
mainfrom
task/#692_startup_refactor_phase2
Sep 1, 2026
Merged

StuartFerguson merged 1 commit into
mainfrom
task/#692_startup_refactor_phase2

Conversation

@StuartFerguson

Copy link
Copy Markdown
Member

Summary

  • Replace ShellNavigationService IServiceProvider page resolution with an injected page factory.
  • Register the transient MyAccountAddressesPage factory in the UI composition root.
  • Preserve transient page creation and existing navigation behavior.

Issue

Closes #692

Verification

  • git diff --check passes.
  • Application compilation emitted the Windows target DLL.
  • Full packaging was not completed because the MAUI packaging step stalled; a compile-only retry was blocked by Windows SDK access permissions.
  • Existing repository warnings remain.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 40 complexity · 0 duplication

Metric Results
Complexity 40
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@StuartFerguson
StuartFerguson merged commit 79613ea into main Sep 1, 2026
12 checks passed
@github-actions
github-actions Bot deleted the task/#692_startup_refactor_phase2 branch November 1, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Phase 2: Remove page-level service location

1 participant