Skip to content

migrate to app_links package - #3683

Draft
malik1004x wants to merge 2 commits into
devfrom
cw-1233-move-to-app-links
Draft

malik1004x wants to merge 2 commits into
devfrom
cw-1233-move-to-app-links

Conversation

@malik1004x

@malik1004x malik1004x commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Issue Number (if Applicable): Fixes #

Description

Please include a summary of the changes and which issue is fixed / feature is added.

Pull Request - Checklist

  • Initial Manual Tests Passed
  • Double check modified code and verify it with the feature/task requirements
  • Format code
  • Look for code duplication
  • Clear naming for variables and methods
  • Manual tests in accessibility mode (TalkBack on Android) passed

Note

Medium Risk
Deep-link entry behavior changes with a new package and no explicit initial-URI handling on launch, which can affect wallet navigation from external links.

Overview
Replaces the forked uni_links dependency with app_links 7.0.0 and wires deep links through AppLinks().uriLinkStream in Root.initUniLinks() (method name unchanged).

Link listeners are simplified by passing handleDeepLinking directly to both the app-link stream and quickActionsStream. The previous getInitialUri() call on startup is removed, so cold-start deep links may no longer be processed unless app_links delivers them another way.

Reviewed by Cursor Bugbot for commit 2e60e82. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2c48681. Configure here.

Comment thread lib/src/screens/root/root.dart Outdated

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