Skip to content

Phase 3: Refactor ShellNavigationService to use DI - #716

Merged
StuartFerguson merged 1 commit into
mainfrom
task/#693_startup_refactor_phase3
Sep 1, 2026
Merged

StuartFerguson merged 1 commit into
mainfrom
task/#693_startup_refactor_phase3

Conversation

@StuartFerguson

Copy link
Copy Markdown
Member

Summary

  • Replace the MyAccountAddressesPage-specific DI factory with Shell route navigation.
  • Remove the unused factory dependency and registration.
  • Preserve constructor injection through the existing registered route.

Verification

  • Focused DI/navigation checks passed.
  • 291 business-logic tests passed.
  • Full mobile build is environment-blocked by local Android/Windows SDK path permissions.

Closes #693

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
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 eb820e9 into main Sep 1, 2026
12 checks passed
@github-actions
github-actions Bot deleted the task/#693_startup_refactor_phase3 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 3: Refactor ShellNavigationService to use DI

1 participant