Skip to content

feat: unify PnP and AutoParent first-time login into single PnP flow#836

Merged
PeterJhongLinksys merged 1 commit into
dev-1.2.11from
feat/unify-pnp-autoparent-747
Apr 29, 2026
Merged

feat: unify PnP and AutoParent first-time login into single PnP flow#836
PeterJhongLinksys merged 1 commit into
dev-1.2.11from
feat/unify-pnp-autoparent-747

Conversation

@HankYuLinksys

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the separate AutoParent First-Time Login route and redirect logic, consolidating both AutoParent and PreConfigured devices into the existing PnP initialization flow
  • Delete autoParentFirstLoginRoute, autoParentFirstLoginStateProvider, and related routing branches in RouterNotifier
  • Clean up unused imports (AutoConfigurationSettings, AutoParentFirstLoginView) and the LocalWhereToGo.firstTimeLogin enum case

Motivation

Previously, AutoParent devices had a dedicated first-time login path that diverged from the standard PnP flow. This added routing complexity without functional benefit — both paths ultimately lead to the same setup experience. Unifying them simplifies the router logic and reduces maintenance surface.

Test plan

  • AutoParent device: verify first-time setup goes through PnP flow correctly
  • PreConfigured device: verify PnP flow is unchanged
  • Returning user (AutoParent, userAcknowledgedAutoConfiguration = true): verify normal login flow
  • Deep links and direct navigation: confirm no broken routes

🤖 Generated with Claude Code

…747)

Remove the separate AutoParent First-Time Login path so both AutoParent
and PreConfigured devices use the same PnP initialization flow.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@PeterJhongLinksys
PeterJhongLinksys merged commit 13a1351 into dev-1.2.11 Apr 29, 2026
@PeterJhongLinksys
PeterJhongLinksys deleted the feat/unify-pnp-autoparent-747 branch April 29, 2026 02:31
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.

3 participants