Skip to content

Conversation

@marktech0813
Copy link

I found the handler and implemented the fix: same-domain navigations now return shouldHaltWebNavigation when an app link is launched.

  • Change made in app/src/main/java/com/duckduckgo/app/browser/applinks/AppLinksHandler.kt
  • In the same-domain branch, after launching the app link, the function now returns shouldTrigger || shouldHaltWebNavigation instead of always returning false unless shouldTrigger was true.

Summary
Updated DuckDuckGoAppLinksHandler.handleAppLink to consistently respect shouldHaltWebNavigation on subsequent same-domain app links, while preserving alwaysTriggerList behavior.

Contribution by Gittensor, learn more at https://gittensor.io/

…app links after first trigger> duckduckgo#7097

I found the handler and implemented the fix: same-domain navigations now return shouldHaltWebNavigation when an app link is launched.
Updated DuckDuckGoAppLinksHandler.handleAppLink to consistently respect shouldHaltWebNavigation on subsequent same-domain app links, while preserving alwaysTriggerList behavior.

Contribution by Gittensor, learn more at https://gittensor.io/
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